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

Amend Gafi Milestone 3 #1279

Merged
merged 1 commit into from
Nov 21, 2022
Merged

Amend Gafi Milestone 3 #1279

merged 1 commit into from
Nov 21, 2022

Conversation

mutobui
Copy link
Contributor

@mutobui mutobui commented Nov 17, 2022

Project Abstract

Please replace these instructions with a brief description of your project summarising key points (1-2 paragraphs).

If your application is a follow-up to a previous grant, please mention which one in the first line of the abstract and include a link to previous pull requests if applicable.

Grant level

  • Level 1: Up to $10,000, 2 approvals
  • Level 2: Up to $30,000, 3 approvals
  • Level 3: Unlimited, 5 approvals (for >$100k: Web3 Foundation Council approval)

Application Checklist

  • The application template has been copied and aptly renamed (project_name.md).
  • I have read the application guidelines.
  • A BTC, Ethereum (USDT/USDC/DAI) or Polkadot/Kusama (aUSD) address for the payment of the milestones is provided inside the application.
  • The software delivered for this grant will be released under an open-source license specified in the application.
  • The initial PR contains only one commit (squash and force-push if needed).
  • The grant will only be announced once the first milestone has been accepted (see the announcement guidelines).
  • I prefer the discussion of this application to take place in a private Element/Matrix channel. My username is: @stefan_muto:matrix.org (change the homeserver if you use a different one)

@mutobui
Copy link
Contributor Author

mutobui commented Nov 17, 2022

Hey W3F team,
We would like to amend milestone 3 by removing pallet_dao from delivery and lowering the cost by one-third 💪

Copy link
Collaborator

@Noc2 Noc2 left a comment

Choose a reason for hiding this comment

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

Thanks for the update. I’m happy to mark it as ready for review.

@Noc2 Noc2 added ready for review The project is ready to be reviewed by the committee members. amendment This PR proposes changes to an existing application. labels Nov 17, 2022
Copy link
Contributor

@keeganquigley keeganquigley left a comment

Choose a reason for hiding this comment

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

thanks @mutobui, are you removing this deliverable because you plan to implement a DAO pallet that already exists?

@mutobui
Copy link
Contributor Author

mutobui commented Nov 18, 2022

Hi @keeganquigley,
Thanks for the question
pallet_dao in the first place is designed for the game projects can create their own DAO, and in the development process, we find out this feature is better to implement at the DApps level (Solidity smart-contract). Thus, we want to remove it from the deliverable.

Copy link
Contributor

@keeganquigley keeganquigley left a comment

Choose a reason for hiding this comment

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

thanks for the explanation!

@semuelle semuelle merged commit fb2af60 into w3f:master Nov 21, 2022
@semuelle
Copy link
Member

For reference, discussion here: w3f/Grant-Milestone-Delivery#616

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
amendment This PR proposes changes to an existing application. ready for review The project is ready to be reviewed by the committee members.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants