Giveth / giveth-dapps-v2

This project is the aggregation of GIVeconomy and Giveth.io DApps in a single repo
https://staging.giveth.io
GNU General Public License v3.0
64 stars 33 forks source link

FE changes to show recurring donations for QF #4222

Open divine-comedian opened 5 months ago

divine-comedian commented 5 months ago

Some FE changes should be made that show this recurring donation be in a QF round like in donate page and donation success page

image.png image.png

Originally posted by @maryjaf in https://github.com/Giveth/giveth-dapps-v2/issues/3284#issuecomment-2122543938

On the donate page for recurring donations - if the project is in a QF round and optimism is an eligible network for the current QF round we should show the QF related project card on the side instead of the regular card.

On the success page we should show the section related to QF matching on this view as well.

maryjaf commented 5 months ago

image

cc: @divine-comedian @MohammadPCh

divine-comedian commented 5 months ago

nice catch MJ! So the value of the recurring donation is not being considered in the "amount raised this round" field on the project card when the project is inside of an active QF round.

We should count the value of eligible recurring donations made inside the QF round to the given project and include it in this number.

@MohammadPCh I can make a new issue if you prefer or track it here

MohammadPCh commented 5 months ago

@divine-comedian Yes, please create a new issue :D

divine-comedian commented 5 months ago

@maryjaf I created a new issue for the scenario you reported - #4287

Can you confirm if the original parts of this issue are completed?

maryjaf commented 5 months ago

like in donate page and donation success page

Donate page:

https://github.com/Giveth/giveth-dapps-v2/assets/111529185/63711a1c-2a86-4ab0-8c54-4b9f9f042d0f

https://github.com/Giveth/giveth-dapps-v2/assets/111529185/011ba277-f7ef-4019-81a5-676a67e8924b

Donation success page:

There is no problem in donation success page image

divine-comedian commented 5 months ago

I don't think we defined how to get the estimated matching for a recurring donation...

My guess to do the estimated matching we would need to know

  1. How many seconds are left in the QF round
  2. How much the user is proposing to donate per second

Then we would need to take the seconds left in QF round, multiplied by amount to donate per second, this would become the value we use for the estimated matching.

maryjaf commented 5 months ago

I don't think we defined how to get the estimated matching for a recurring donation...

My guess to do the estimated matching we would need to know

  1. How many seconds are left in the QF round
  2. How much the user is proposing to donate per second

Then we would need to take the seconds left in QF round, multiplied by amount to donate per second, this would become the value we use for the estimated matching

I don't get it correctly , so this behavior is expected or it should be changed

  • [ ] 1- There is difference between one time and recurring tab for project in a round, please take a look @MohammadPCh
  • [ ] 2- Estimated matching isn't shown in recurring donation tab
divine-comedian commented 5 months ago

Oh, the 1st issue looks to be a bigger issue, not related to this issue - I would consider it out of scope here and we can follow it up elsewhere and see how Lauren wants to handle it.

The second issue however - looks like the QF project card hasn't been implemented for recurring donations yet.

MohammadPCh commented 5 months ago

@divine-comedian can we close this issue? or wrap up the remaining tasks?

divine-comedian commented 4 months ago

@divine-comedian can we close this issue? or wrap up the remaining tasks?

Did you resolve the second issue MJ reported? The QF version of the project card should show up when user selects to make a recurring donation from donate page

MohammadPCh commented 3 months ago

@lovelgeorge99, could you take care of this and resolve this old issue? Thanks!

lovelgeorge99 commented 3 months ago

For this one i am waiting for a PR in imapct graph to get reviewd which gets the recurring dontaions from a data range.

@lovelgeorge99, could you take care of this and resolve this old issue? Thanks!

divine-comedian commented 2 weeks ago

@lovelgeorge99 what is the update on this issue?

lovelgeorge99 commented 2 weeks ago

@divine-comedian this issuse was resolved here https://github.com/Giveth/giveth-dapps-v2/issues/3284

divine-comedian commented 2 weeks ago

Looks like MJs reported issue is done - moving this issue over to done.

image