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

Add contribution to Proof_Of_ContributionSnapshot4.md #222

Closed
wants to merge 19 commits into from

Conversation

ArtemUpnode
Copy link
Contributor

@ArtemUpnode ArtemUpnode commented May 1, 2024

Note

UPDATE 30.05:
Updated the weights based on Snapshot 5: reduced by 35% based on the new Implied value of a Weight and added 6% for contribution this month. Resubmitted PR for Snaphot 5: #287.

Added contributions for:

And additionally fixed amm broken link in this PR.

@ArtemUpnode ArtemUpnode changed the title fix: broken amm link Add contribution to Proof_Of_ContributionSnapshot4.md May 1, 2024
@ArtemUpnode
Copy link
Contributor Author

This PR was not closed or merged, so it was resubmitted for Snapshot 5: #287.

@DavidAJohnston
Copy link
Contributor

The MRC is not accepted, developed or delivered yet. Please re submit next month.
The rest of these documents have conflicts.
I'd suggest making much smaller pull requests with less content to be in conflict and for easier review. Thanks.

@ArtemUpnode
Copy link
Contributor Author

The MRC is not accepted, developed or delivered yet. Please re submit next month. The rest of these documents have conflicts. I'd suggest making much smaller pull requests with less content to be in conflict and for easier review. Thanks.

Thanks for the advice. Can you please clarify which conflicts you are referring to?

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.

2 participants