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

The Secretary Program #306

Closed

Conversation

Doordashcon
Copy link
Contributor

@Doordashcon Doordashcon commented May 9, 2024

Closes polkadot-fellows/help-center#10

On Chain Structure

The on-chain program consists of one rank and the default Candidate:

  • Secretary(rank 1) with the corresponding Origin(Secretary), which represents the collective voice of members of that rank.

Referendum

The SecretaryReferenda instance of referenda pallet consists of one track, corresponding to its Origin and invites all members to vote.

Membership Management

Governing parameters(i.e active_salary, passive_salary, demotion_period e.t.c.) are set by FellowshipAdmin and Fellows(DAN III) Origins.

Members themselves are without the power to promote or demote, this right is reserved to token holders via Polkadot OpenGov and Fellows(DAN III).

Sub-Treasury

The program operates its own sub-treasury, managed by OpenGov.

Fellowship Program Referendum

The Secretary Program can submit proposals on behalf of the fellowship program on any track.

Copy link

Review required! Latest push from author must always be reviewed


pub type SecretaryCoreInstance = pallet_core_fellowship::Instance3;

impl pallet_core_fellowship::Config<SecretaryCoreInstance> for Runtime {
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we do not need this core pallet. It does passive demotions and lets to setup some requirements for promotions and retentions. I do not thing we need this for the Secretary for now with a single member.
So I think inductions/promotions/demotions can be done directly through the ranked collective pallet.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, Sounds good.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This would require implementing the upgraded ranked collective pallet, with the inclusion of AddOrigin and RemoveOrigin.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The core pallet is also used for MemberSwappedHandler and I think we should leave the core pallet in.

The SecretaryCollective is bound to have more than one member in the coming months as developments progress.

@Doordashcon
Copy link
Contributor Author

Doordashcon commented Jun 10, 2024

Moving this to a new branch #347

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.

Add a secretary collective
2 participants