-
Notifications
You must be signed in to change notification settings - Fork 88
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
Fix set role admin #879
Fix set role admin #879
Conversation
Temporarily disable for pull requests until limiting rate API issue is resolved. Reference: ubiquity#876
Temporarily disable for pull requests until limiting rate API issue is resolved. Reference: ubiquity#876
…age-changes-workflow-to-manual chore: temporarily disable storage changes workflow for pull requests
0bdc302
to
3e96286
Compare
closing reopening? 😆 |
@molecula451 I am sorry for the spam. I was confused and realized too late that we disabled GraphQL and contract storage changes workflows in development branch, but fix-review branch still has it enabled. So it is enabled and still failing here.. |
yeah you pointed the disabled to the development branch lol |
The setRoleAdmin can be only accessed by the admin.
3e96286
to
a9df0e2
Compare
doesn't look clean, the PR is gathering my commits to this PR, better just re-do the disable |
You are right. Let me clean up this now. I will delete the pr, merge development to fix-review. Then base my pull request on the updated fix-review. |
Resolves sherlock-audit/2023-12-ubiquity-judging#90