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

FOR REVIEW: RPIP34 (VOPs Framework) Update #94

Merged
merged 4 commits into from Nov 9, 2023

Conversation

LongForWisdom
Copy link
Contributor

Updated based on feedback from Val. I would like to get some feedback from community before finalizing. So this should remain a as draft for now.

Things updated:

  • Unified examples around the EMC (example management committee) for clarity.
  • Clarified elements fields some.
  • Moved templates to implementation section.
  • Moved backwards compatibility (= changes to existing processes) to specification section.
  • Added pDAO budget management to VOPs framework.
  • Explicitly supporting removing or modifying VOP definitions.

Major outstanding:

  • Get community feedback.
  • Create a separate PR containing the required modifications to other RPIPs in order to integrate them into this framework.

@LongForWisdom LongForWisdom changed the title DRAFT: RPIP34 (VOPs Framework) Update FOR REVIEW: RPIP34 (VOPs Framework) Update Nov 9, 2023
@LongForWisdom
Copy link
Contributor Author

@Valdorff (and other RPIP editors) I would appreciate if you'd merge these changes to the draft, as I'm no longer actively working on this. Probably doesn't need to become stagnant just yet, but your choice.

Copy link
Collaborator

@Valdorff Valdorff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merging draft as is. Understood it may not get immediate attention or may even get closed/stagnant eventually.

@Valdorff Valdorff merged commit d3dddcf into rocket-pool:main Nov 9, 2023
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.

None yet

2 participants