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 quadratic funding pallet milestone-1 #104

Merged
merged 1 commit into from
Apr 5, 2021

Conversation

zhangjiannan
Copy link
Contributor

@zhangjiannan zhangjiannan commented Mar 1, 2021

Milestone Delivery Checklist

  • The milestone-delivery-template.md has been copied and updated.
  • The invoice form 📝 has been filled out for this milestone.
  • This pull request is done by the same account, which is responsible for the pull request of the accepted application.
  • In the case of acceptance, the payment will be transferred to the initial BTC payment address.
  • The delivery is according to the milestone delivery guidelines.

@zhangjiannan
Copy link
Contributor Author

How can I pass the checks?

@semuelle
Copy link
Member

semuelle commented Mar 1, 2021

The check is just a bot writing your delivery data into a database. No need to worry about it.

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 again for the delivery. I started to look into it and have a few questions/requests:

@zhangjiannan
Copy link
Contributor Author

@Noc2 thanks for your feedback. We have made changes in the repo.

  • updated readme
  • updated unit tests in test.rs
  • fixed warnings and glitches
  • provided in-line documentation
  • added start_round function and now the pallet supports multiple rounds

@Noc2
Copy link
Collaborator

Noc2 commented Mar 12, 2021

Sorry for the late response. This looks much better. Just a quick follow-up: Could you still provide a small testing guide or tutorial? I quickly tried to play with it via the extrinsics tab of polkadot.js, but for example didn’t immediately know how the Admin set up works (or which accounts I need to use to start a round). And could you point me to line in the code that calculates the matched amounts or payment to each project based on quadratic funding? (You can find a python implementation here: https://github.com/gitcoinco/quadratic-funding)

@zhangjiannan
Copy link
Contributor Author

@Noc2 Thanks for feedback. We are not exactly following GitCoin's approach, we are mostly migrating a decentralized version of quadratic funding grant currently running on HackerLink (see BSC Grant Round-1 https://hackerlink.io/en/Grant/BSC/Round/1/detail), and the algorithm follows Vitalik's original paper (https://vitalik.ca/general/2019/12/07/quadratic.html). The fund distribution happens within fn end_round(), and it's based on each project's support area within a round. We will provide more documentation and in-line comments to help navigate shortly.

@mmagician
Copy link
Contributor

I'm putting this temporarily on hold due to parallel participation in the hackathon. The hackathon organisers will contact you regarding further steps.

@mmagician mmagician added on hold and removed on hold labels Mar 17, 2021
@Noc2
Copy link
Collaborator

Noc2 commented Mar 24, 2021

@zhangjiannan Any update on this. Should I take another look at the delivery

@zhangjiannan
Copy link
Contributor Author

Hi @Noc2 we have developed a frontend and integrated into HackerLink. I will share a demo video to help better understand the pallet soon.

@Noc2
Copy link
Collaborator

Noc2 commented Apr 1, 2021

@zhangjiannan I just took another quick look at it. Looks good, could you also integrate the json for polkadot.js into the testing guide: https://github.com/zhangjiannan/QFgrant/blob/main/doc/test-guide.md?

@zhangjiannan
Copy link
Contributor Author

Hi @Noc2 the json has been added to decode round and project . We deployed a node for testing, you can access it using custom endpoint wss://dao.tophacker.com.

@Noc2
Copy link
Collaborator

Noc2 commented Apr 5, 2021

I’m happy to tell you that the milestone is a pass. You can find my evaluation here. However, I noticed that the payment address on the invoice is different from the original contract. Could you either amend the payment address of the contract or send us a new invoice?

@Noc2 Noc2 merged commit 6e54670 into w3f:master Apr 5, 2021
@Noc2
Copy link
Collaborator

Noc2 commented Apr 16, 2021

failfmi pushed a commit to LimeChain/Grant-Milestone-Delivery that referenced this pull request Sep 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants