-
Notifications
You must be signed in to change notification settings - Fork 15
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: "Upgrade" button on Calmity Banner should be enabled for all users #3520
Fix: "Upgrade" button on Calmity Banner should be enabled for all users #3520
Conversation
@iamsamgibbs Thank you for picking this up, sorry for the confusion on this issue as you have done the work already. @melyndav happened to not have Reputation enabled when running into this originally. However, hiding the banner completely for everyone but root is not the desired behavior. As it could pose a security risk, and everyone should be informed of a new version. We already handle permissions in the Upgrade colony version action, and we also allow a colony to be upgraded using Reputation. So, the correct behavior here should be that the upgrade button is never disabled, and should always open the "Upgrade colony version" action. Acceptance criteria
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As above.
18e8a9c
to
4d89f1e
Compare
No problem Arren! Updated to match the new specs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@iamsamgibbs Thank you, and sorry again for the mess around. Looks good to me :)
Description
This PR ensures the "Upgrade" button on the Calamity Banner is always enabled for all users when a new colony version is available.
Testing
To save you the hassle of deploying a colony on an older version, we must first update the
canColonyBeUpgraded
function insrc/utils/checks/canColonyBeUpgraded.ts
to return true.Diffs
Deletions ⚰️
canUpgradeColony
and related uses fromuseCalamityBannerInfo
Resolves #3340