BuidlGuidl / grants.buidlguidl.com

https://grants-bg.vercel.app
MIT License
3 stars 3 forks source link

Review stats logic #41

Closed Pabl0cks closed 8 months ago

Pabl0cks commented 8 months ago

Checking stats after initial grants data load, numbers felt a bit strange. I think we're showing completedGrants as Total Grants.

Creating this issue to discuss about the stats to set a initial logic for the section.

Should we maybe change "Submissions" for "Completed Grants"? Or delete "Submissions" and show "Completed Grants" and "Active Grants" and people may not need "Total Grants" stat? Just thinking out loud

technophile-04 commented 8 months ago

Tysm @Pabl0cks, agree with all the pointers and created a #42 for it, something Carlos as discussed.

ETH Granted. Should we save this stat in the grants data and query to it? Right now we are showing the sum of askAmount for completed grants. This stat could be updated after we transfer the grant amount.

I think askAmount is something that is something going to be changed in the future. Currently, the assumption is ETHGranted == askAmount. But yup maybe we should add a field ethGranted to grants schema

So, askAmount => The amount builder selected while submitting new grant proposal ethGranted => This will change two times in the lifecycle of the grant :

  1. When admins mark grant as "approved" and sends 50% of askAmount
  2. When admins mark grant as "completed" and sends rest 50% of askAmount

But cc @carletex and would love to hear his thoughts 🙌

carletex commented 8 months ago

42