search

NEAR-DevHub / neardevhub-contract

DevHub Portal Product Contract (Hosted on NEAR Blockchain) – Includes other instances (e.g. Infrastructure, Events)
https://neardevhub.org
19 stars 15 forks source link

Update to near-sdk-rs 4.1.1 #63

Closed PiVortex closed 1 year ago

PiVortex commented 1 year ago

Solution for https://github.com/near/neardevhub-contract/issues/23

PiVortex commented 1 year ago

All $ cargo test tests passed These instructions for testing have been followed, and there seem to be no issues.

My solution for the JSON arguments flipping order here is rather messy, it is likely to cost more gas and adds somewhat of a useless dependency in the toml. I tried to fix it with a raw string, but that solution was also messy and didn't work.

Also, shouldn't admins be able to make the sort of changes in lib.rs not just moderators?

frol commented 1 year ago

@PiVortex Well, at the end of the day, it turned out that the access control rules had a user with invalid account id (Cindytest.near), so I had to revert the contract once again, debug it, remove the user, and deploy this new version of the contract back. Now it is up and running, thanks for contributing it!