-
Notifications
You must be signed in to change notification settings - Fork 81
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
Find out backward compatibility issue #114
Comments
The guys from SQA will take care of this issue. |
The command
Fails and talking to @ctindogarus4f it has been suggested it can be only run on mac-os, unfortunately most of our teams runs on Ubuntu Linux |
@GentBinakuSQA I had the same issue, created a PR near/near-cli#938 - hopefully it gets merged or you can use |
@ctindogarus4f using the scripts i created in PR #111 - I was able to test the recently merged #115 and confirm that backward compat is now working. Doing the final test of checking proposals does not have Borsh deserialize issues. |
Problem:
There was a backward compatibility error introduced at some point in the sputnik contracts and we need to find out the exact commit which introduced it.
How to find it?
Basically by picking random commits from https://github.com/near-daos/sputnik-dao-contract/commits/main and see where the error was introduced.
How to do it?
You need to follow the instructions below.
Instructions:
ctindogaru.testnet
with your own testnet account.xsda21da
)cargo clean
and wait for completion.cargo update
and wait for completion../build
and wait for completion.near view ctindogaru-dao.sputnik-factory.ctindogaru.testnet get_proposal '{"id": 0}’
. Please make sure to replacectindogaru.testnet
with your own testnet account.Cannot deserialize value with Borsh
error, it means that commit id contains a backward compatible issue. Please create a new DAO by following these instructions, but make sure to replacectindogaru-dao
with another name andctindogaru.testnet
with your own testnet account.:Repeat instructions 2-8 until the errror does not show up anymore and you find the exact commit which introduced this backward compatible issue.
10. If you run out of funds in your wallet, please create a new account by typing
near login
and repeat instructions 1-9.The text was updated successfully, but these errors were encountered: