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

ammend iris grant prop #812

Merged
merged 3 commits into from
Feb 15, 2022
Merged

ammend iris grant prop #812

merged 3 commits into from
Feb 15, 2022

Conversation

driemworks
Copy link
Contributor

@driemworks driemworks commented Feb 5, 2022

#657

For which grant level are you applying?

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

@driemworks driemworks mentioned this pull request Feb 5, 2022
9 tasks
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 amendment. I noticed that you also reduced the FTE and Estimated Duration for milestone 2 and 3. Could you potentially also reduce the price accordingly?

@Noc2 Noc2 self-assigned this Feb 7, 2022
@Noc2 Noc2 added the changes requested The team needs to clarify a few things first. label Feb 7, 2022
@driemworks
Copy link
Contributor Author

Thanks for the amendment. I noticed that you also reduced the FTE and Estimated Duration for milestone 2 and 3. Could you potentially also reduce the price accordingly?

I've updated the document. For milestone two, the correct estimate is 6 weeks, not 4 weeks. I left the cost for M2 unchanged since it is a considerable effort and delivers something new within the substrate framework (Proof of Authority with validators incentivized to provide storage, and also a naive scheme for determining value of data). For milestone three (the result of which is a functional use case), I've updated the cost. Let me know if that seems reasonable to you.

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.

Sounds good to me. Could you maybe also update the total costs? Just so it doesn’t lead to any confusion.

@driemworks
Copy link
Contributor Author

Absolutely. It's updated now.

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. I will share it with the rest of the team.

@Noc2 Noc2 added amendment This PR proposes changes to an existing application. and removed changes requested The team needs to clarify a few things first. labels Feb 8, 2022
@driemworks
Copy link
Contributor Author

Just sending a little nudge as I am approaching the completion of milestone 2.

@Noc2
Copy link
Collaborator

Noc2 commented Feb 14, 2022

Thanks for chasing us. I will share it again with the team.

Copy link
Contributor

@alxs alxs left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Collaborator

@takahser takahser left a comment

Choose a reason for hiding this comment

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

LGTM as well.

@Noc2 Noc2 merged commit cf56030 into w3f:master Feb 15, 2022
@driemworks driemworks mentioned this pull request May 18, 2022
18 tasks
@driemworks driemworks mentioned this pull request Jun 16, 2022
18 tasks
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants