Skip to content
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

Feat: Enable and Disable Proxy Colonies via the Reputation Motion #4063

Draft
wants to merge 2 commits into
base: feat/3450-add-proxy-colony-motion
Choose a base branch
from

Conversation

rumzledz
Copy link
Contributor

@rumzledz rumzledz commented Jan 9, 2025

Description

Note

This PR only tackles the sagas for motions. Any form-specific logical issues you see on this PR is not introduced by this PR and will be dealt with separately

This PR handles the enabling and disabling of Proxy Colonies via the Reputation motion.

Please test this with block-ingestor PR #313: Feat: Enable and Disable Proxy Colonies via the Reputation Motion

enable-disable-proxy-colony

Testing

Note

This PR only tackles the sagas for motions. Any form-specific logical issues you see on this PR is not introduced by this PR and will be dealt with separately

  1. Install the Reputation extension
  2. On the Reputation extension's config page, select "Custom (Advanced)
  3. Set the following fields to 0.005 (this will make your life easier)
  • Staking Phase Duration
  • Voting Phase Duration
  • Reveal Phase Duration
  1. Click Manage Colony on the Sidebar
  2. Scroll the Action Form down and click on Manage supported chains
  3. Set the Add/remove field to Add chain support
  4. Set the Chain field to Ethereum
  5. Set the Decision method field to Reputation
  6. Fill in all other required fields
  7. Submit the form
  8. Approve the motion and fully stake it
  9. Wait for the motion countdown to end
  10. Finalise the motion
  11. Verify that the next time you bring up the Manage supported chains form:
  • You are not able to select Ethereum from the Chains list when adding chain support
  • You are able to select Ethereum from the Chains list when removing chain support

Resolves #3457

@rumzledz rumzledz self-assigned this Jan 9, 2025
@rumzledz rumzledz force-pushed the fix/3457-enable-disable-proxy-colonies-via-reputation branch from 9f5e8ef to f3dc86e Compare January 9, 2025 19:08
@rumzledz rumzledz changed the title Feat: Enable and Disable Proxy Colonies via a Motion Feat: Enable and Disable Proxy Colonies via the Reputation Motion Jan 9, 2025
@rumzledz rumzledz force-pushed the fix/3457-enable-disable-proxy-colonies-via-reputation branch from f3dc86e to 220760c Compare January 10, 2025 12:46
@rumzledz rumzledz marked this pull request as ready for review January 10, 2025 13:02
@rumzledz rumzledz requested a review from a team as a code owner January 10, 2025 13:02
@rumzledz rumzledz marked this pull request as draft January 10, 2025 14:27
@rumzledz rumzledz marked this pull request as ready for review January 13, 2025 10:15
@rumzledz rumzledz marked this pull request as draft January 13, 2025 10:19
@rumzledz rumzledz force-pushed the fix/3457-enable-disable-proxy-colonies-via-reputation branch from b50a8b3 to 470c1f2 Compare January 13, 2025 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant