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

Include Recurring donation number on QF project card #4287

Closed
divine-comedian opened this issue Jun 17, 2024 · 17 comments
Closed

Include Recurring donation number on QF project card #4287

divine-comedian opened this issue Jun 17, 2024 · 17 comments
Assignees
Labels
QF Quadratic Funding Related

Comments

@divine-comedian
Copy link
Contributor

divine-comedian commented Jun 17, 2024

It would be good if we consider this scenario

  • The project hasn't any one time donation in a round
  • The project has a recurring donation and it be counted in estimated matching
  • The project card is shown like below pic, amount raised in this round is 0 , estimated matching is more than zero (because of recurring donation in the round)

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.

image

Originally posted by @maryjaf in #4222 (comment)

FYI @koday1

@MohammadPCh
Copy link
Collaborator

@lovelgeorge99 can you take it, please? Thanks

@maryjaf
Copy link
Collaborator

maryjaf commented Aug 12, 2024

When I change the start time of round , the sum of recurring donation in the round should be changed, am I right?

  • the start date of round was 19/06
  • the raised amount in the round was 129.63
  • I changed the start time into 10/08
  • but raised amount hasn't been changed and it is shown 129.63

https://staging.giveth.io/project/testtttttttt?tab=donations

and also the Number of contributor should be set based on both one time and recurring donation
for example for above project it should be set 2 contributors

@maryjaf
Copy link
Collaborator

maryjaf commented Aug 12, 2024

and one more thing this issue hasn't been tested but I think since this change was on stagimg it was deployed on prod by the previous release, am I right?

unfortunately I hadn't seen this in QA board yesterday

@lovelgeorge99
Copy link
Collaborator

lovelgeorge99 commented Aug 12, 2024

@maryjaf the current changes that I made , should reflect the change of amount in the project card . I have not made the changes to reflect it on the project donation section ..will work on that next. Thanks for the feedback

@maryjaf
Copy link
Collaborator

maryjaf commented Aug 12, 2024

When I change the start time of round , the sum of recurring donation in the round should be changed, am I right?

  • the start date of round was 19/06
  • the raised amount in the round was 129.63
  • I changed the start time into 10/08
  • but raised amount hasn't been changed and it is shown 129.63

https://staging.giveth.io/project/testtttttttt?tab=donations

and also the Number of contributor should be set based on both one time and recurring donation for example for above project it should be set 2 contributors

In this scenario, the project card is shown like below pic

image

it seems the number of amount raised in the round and estimated matching isn't correct

@maryjaf
Copy link
Collaborator

maryjaf commented Aug 12, 2024

and one more thing this issue hasn't been tested but I think since this change was on stagimg it was deployed on prod by the previous release, am I right?

unfortunately I hadn't seen this in QA board yesterday

Since this issue hadn't been tested, Can we revert this change on prod ?

@lovelgeorge99
Copy link
Collaborator

and one more thing this issue hasn't been tested but I think since this change was on stagimg it was deployed on prod by the previous release, am I right?
unfortunately I hadn't seen this in QA board yesterday

Since this issue hadn't been tested, Can we revert this change on prod ?

Yes,this should be reverted

@maryjaf
Copy link
Collaborator

maryjaf commented Aug 13, 2024

and one more thing this issue hasn't been tested but I think since this change was on stagimg it was deployed on prod by the previous release, am I right?
unfortunately I hadn't seen this in QA board yesterday

Since this issue hadn't been tested, Can we revert this change on prod ?

Yes,this should be reverted

who should do that ?
@lovelgeorge99
cc: @MohammadPCh @divine-comedian

@MohammadPCh
Copy link
Collaborator

@maryjaf @lovelgeorge99 Could we explore fixing it rather than reverting?

@MohammadPCh
Copy link
Collaborator

@maryjaf @lovelgeorge99 I'll remove the recurring donation amount from the card and release a hotfix. Please follow up on this issue to ensure it's fully resolved. Thanks!

@lovelgeorge99
Copy link
Collaborator

lovelgeorge99 commented Aug 21, 2024

@maryjaf i have added the sum of recurring donations and the number of donars made in QF round
image

also should that amount also be included here?
image

@divine-comedian divine-comedian removed P2 A normal to do item or feature. Product Backlog labels Aug 22, 2024
@maryjaf
Copy link
Collaborator

maryjaf commented Aug 25, 2024

When I change the start time of round , the sum of recurring donation in the round should be changed, am I right?

  • the start date of round was 19/06
  • the raised amount in the round was 129.63
  • I changed the start time into 10/08
  • but raised amount hasn't been changed and it is shown 129.63

https://staging.giveth.io/project/testtttttttt?tab=donations

and also the Number of contributor should be set based on both one time and recurring donation for example for above project it should be set 2 contributors

Have you seen this scenario ? @lovelgeorge99

@mohammadranjbarz
Copy link
Contributor

When I change the start time of round , the sum of recurring donation in the round should be changed, am I right?

  • the start date of round was 19/06
  • the raised amount in the round was 129.63
  • I changed the start time into 10/08
  • but raised amount hasn't been changed and it is shown 129.63

https://staging.giveth.io/project/testtttttttt?tab=donations

and also the Number of contributor should be set based on both one time and recurring donation for example for above project it should be set 2 contributors

We connect the qfRound and recurring donation streams when donations are made, so changing the start time of the qfRound later would not affect it and can't change the relation between donations and qfRounds

@lovelgeorge99
Copy link
Collaborator

image
@mohammadranjbarz this is the recurring donations in that qf round and this should be shown in the QF project card...right?

@mohammadranjbarz
Copy link
Contributor

image @mohammadranjbarz this is the recurring donations in that qf round and this should be shown in the QF project card...right?

@lovelgeorge99 Yeah I think so

@lovelgeorge99
Copy link
Collaborator

image
https://staging.giveth.io/project/testing-project-lovel-george

@maryjaf This project does not have any one time donation only recurring donations and it is shown in the QF project card>
Can you check and verify

@maryjaf
Copy link
Collaborator

maryjaf commented Sep 29, 2024

@divine-comedian divine-comedian closed this as completed by moving to Merged to Production in All-Devs Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
QF Quadratic Funding Related
Projects
Status: Done
Development

No branches or pull requests

5 participants