-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update to ink! 5 #298
Update to ink! 5 #298
Conversation
✅ Deploy Preview for inklink canceled.
|
Nice! Had this on my list for refreshing this dapp. However, it's just the contract source which is updated to v5, the actual v3 contract is still deployed and used by the dapp afaik. The contract seems to have a feature to be updated, do we want to upgrade it to v5? Otherwise we dapp still needs the metadata from v3 to work properly as well as polkadotjs is not yet ready for v5. Might lead to crashes if we just upgrade the contract without adapted the frontend with the updated pjs just yet. |
So far it's only deployed on Rococo, so we can just ignore the running instance. I told Sebastian that you would take care of switching the frontend to ink!athon/ |
Update contract to ink5
Update CI
Add tests
Rename constructor from
default
tonew