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

Archived QF round summary page & link on the main QF page #3748

Closed
laurenluz opened this issue Feb 13, 2024 · 37 comments
Closed

Archived QF round summary page & link on the main QF page #3748

laurenluz opened this issue Feb 13, 2024 · 37 comments
Assignees
Labels
Done P2 A normal to do item or feature. QF Quadratic Funding Related

Comments

@laurenluz
Copy link
Member

This is coming from #3674

After a QF round has ended, we want to preserve info about the banner, round dates, stats, projects in the round, etc and give it a unique slug. I think this is already somehow built, @MohammadPCh was working on it.

What we need new is some "homepage" where you can see a list of all archived rounds & their corresponding links.

And, somewhere on the main giveth.io/qf page that links to "See Archived round"... This link should not be super prominant.

Does that cover everything @MohammadPCh ?

@Tosinolawale are you able to help w/ this?

@laurenluz laurenluz added Design P2 A normal to do item or feature. labels Feb 13, 2024
@Tosinolawale
Copy link

This is coming from #3674

After a QF round has ended, we want to preserve info about the banner, round dates, stats, projects in the round, etc and give it a unique slug. I think this is already somehow built, @MohammadPCh was working on it.

What we need new is some "homepage" where you can see a list of all archived rounds & their corresponding links.

And, somewhere on the main giveth.io/qf page that links to "See Archived round"... This link should not be super prominant.

Does that cover everything @MohammadPCh ?

@Tosinolawale are you able to help w/ this?

On it

@laurenluz
Copy link
Member Author

ok Thanks @Tosinolawale - all set.

The most up-to-date version is in figma:
image

@MohammadPCh would love your feedback - think this will work ok?

cc @jainkrati

@MohammadPCh
Copy link
Collaborator

MohammadPCh commented Feb 28, 2024

@laurenluz @Tosinolawale Great design! I have a small suggestion:
Since we have a dedicated page for this at https://giveth.io/qf-archive (maybe it's better to change it to /qf-rounds wdyt?), we can remove the tabs on top.
Additionally, we'll need the backend API for fetching this information.
cc: @jainkrati

@laurenluz
Copy link
Member Author

this is the issue for adding banners to the backend cooresponding to QF rounds: #3665 cc @jainkrati @mohammadranjbarz @MohammadPCh

@laurenluz
Copy link
Member Author

@MohammadPCh I think the tabs are kind of nice... then we could just have giveth.io/qf go to this page, default view showing the active round... and then if you click the archive tab, you get to /qf-archive where it shows you all the past/ended rounds.

@laurenluz
Copy link
Member Author

this will work well if we end up expaning qf to run multiple rounds at once.. because then the active round tab can look similar to the archive tab... but be a homepage for different active rounds.

@maryjaf
Copy link
Collaborator

maryjaf commented Apr 3, 2024

It seems this issue isn't ready for testing and is in a wrong state in the board , am I right? @MohammadPCh

@MohammadPCh
Copy link
Collaborator

@maryjaf It's on progress.

@MohammadPCh
Copy link
Collaborator

@laurenluz Could you please confirm whether the design meets your requirements? I'm eager to begin working on it.

@laurenluz
Copy link
Member Author

Hey @MohammadPCh! The design looks good, I changed some copy and made a small comment about removing "Default" as sort option.... but aside from that, it's good to go!

@MohammadPCh
Copy link
Collaborator

Hey @mohammadranjbarz,

We require this information to be fetched from the backend. Could you please provide it?

We need to retrieve all ended (archived) QF rounds with the following details:

  1. Slug
  2. Name
  3. Matching pool
  4. Total donations amount
  5. Unique donors
  6. Start date
  7. End date

Additionally, we need to sort them by the following criteria:

  • Matching pool
  • Number of unique donors
  • Newest
  • Oldest

Thank you.
Screenshot 2024-04-09 at 1 58 29 PM
cc: @laurenluz @jainkrati

@jainkrati
Copy link
Collaborator

@mohammadranjbarz if your PR has all data, lets merge it else pls assign to @CarlosQ96 to proceed on this

@jainkrati
Copy link
Collaborator

@CarlosQ96 pls build on the PR by @mohammadranjbarz linked above.

@jainkrati
Copy link
Collaborator

@CarlosQ96 pls guide @RamRamez on remaining fix for this.

@jainkrati jainkrati assigned RamRamez and unassigned MohammadPCh Apr 23, 2024
@MohammadPCh
Copy link
Collaborator

This is the final result:
Screenshot 2024-05-12 at 10 30 12 AM
@laurenluz, should we display the dollar sign next to the "Matching Pool"? I remember that the matching pool may have a different unit.
I attempted to retrieve the allocatedTokenSymbol from the backend, but this query doesn't support it.
cc: @jainkrati @RamRamez

@RamRamez
Copy link
Collaborator

@MohammadPCh I added allocatedFundUSD and allocatedTokenSymbol to the query:

  query (
   $limit: Int
   $skip: Int
   $orderBy: QfArchivedRoundsOrderBy
  ) {
    qfArchivedRounds(
      limit: $limit
      skip: $skip
      orderBy: $orderBy
    ) {
      id
      name
      slug
      isActive
      allocatedFund
      allocatedFundUSD
      allocatedTokenSymbol
      eligibleNetworks
      beginDate
      endDate
      totalDonations
      uniqueDonors
    }
  }

@RamRamez
Copy link
Collaborator

@laurenluz you can fill the allocatedFundUSD and allocatedTokenSymbol fields for all archived QF rounds from AdminJS. If you like to show USD value of allocated fund for archived QF rounds, you can fill only allocatedFundUSD and if you want to show the token amount allocated, you can only fill allocatedTokenSymbol field.

@MohammadPCh
Copy link
Collaborator

@RamRamez Thanks

@laurenluz
Copy link
Member Author

laurenluz commented May 21, 2024

@maryjaf - when you test this, can you also check to make sure that things look right and make sense when there is NO active QF round?

We should basically have some simple "there is no active round" and the regular, quadratic funnding generic banner in this case.

related: #4171

@maryjaf
Copy link
Collaborator

maryjaf commented May 23, 2024

  • 1-Should the mid page QF banner be shown in the page of archived QF? @laurenluz
    image

  • 2-"Load more" button isn't shown and all of archived qf round in stg aren't shown in this page

  • 3- I see some kind of irregularity in the columns of this table, do you think the same?
    @MohammadPCh
    STG:
    image
    PRODUCTION
    image

@maryjaf
Copy link
Collaborator

maryjaf commented May 23, 2024

@maryjaf - when you test this, can you also check to make sure that things look right and make sense when there is NO active QF round?

We should basically have some simple "there is no active round" and the regular, quadratic funnding generic banner in this case.

There are lots of archive rounds on stg and I think removing all of them from db is the only way and isn't a good idea because we need this data in our test
@MohammadPCh Have you implemented a design for this case ? and do you think it could be able to tested ?

@laurenluz
Copy link
Member Author

I sent @MohammadPCh updated copy for the mid-page banner text:

Archived QF Rounds
Explore past quadratic funding rounds on Giveth! Check out the projects who participated, matching funds, donations and more info on this page.

Re:

There are lots of archive rounds on stg and I think removing all of them from db is the only way and isn't a good idea because we need this data in our test
@MohammadPCh Have you implemented a design for this case ? and do you think it could be able to tested ?

Maybe there is some confusion? I wasn't suggesting that we remove any archived rounds. Just, when there is not a round that is currently open & live, the "active round" tab... which should be the default for giveth.io/qf...

I think it is not clear & there is not clear design, I will work with @Tosinolawale on it asap.

@laurenluz
Copy link
Member Author

laurenluz commented May 23, 2024

@MohammadPCh can you confirm also where you are getting the value "unique donors"? because this shouldn't be just unique addresses, but should be, effectively:

  • donors who had a passport score above the threshold

  • donors whose donations were within an active QF round, to a QF project, on an eligible for QF chain

  • donors who are not "flagged as sybil for the round" (in our backend, set in admin bro)

  • Also we should have some setting in admin bro where we can mark if a round is completed or not... if the round is not complete, the unique donors should say "pending" cc @CarlosQ96

  • Finally, can we change the copy "Total Donations" to say just "Donations (USD value)"

@laurenluz
Copy link
Member Author

working on fixing the /qf homepage design & edge cases in here: #4171

@MohammadPCh
Copy link
Collaborator

@laurenluz @maryjaf Thanks, guys!
I have updated it.
@laurenluz, regarding the unique donors, I fetched the data from the back-end, so @mohammadranjbarz or @CarlosQ96 can help us with that.
I also made a modification to the table on my own and merge start and end date columns (sorry 😢, but I thought it was necessary):
Screenshot 2024-05-26 at 3 25 27 PM

@maryjaf
Copy link
Collaborator

maryjaf commented May 26, 2024

  • 2-"Load more" button isn't shown and all of archived qf round in stg aren't shown in this page
  • 3- I see some kind of irregularity in the columns of this table, do you think the same?

Thanks @MohammadPCh These 2 items have been resolved.

This item remains @laurenluz @MohammadPCh
[ ] 1-Should the mid page QF banner be shown in the page of archived QF? @laurenluz

@maryjaf
Copy link
Collaborator

maryjaf commented May 29, 2024

  • Also we should have some setting in admin bro where we can mark if a round is completed or not... if the round is not complete, the unique donors should say "pending" cc @CarlosQ96

@MohammadPCh can you confirm also where you are getting the value "unique donors"? because this shouldn't be just unique addresses, but should be, effectively:

  • donors who had a passport score above the threshold
  • donors whose donations were within an active QF round, to a QF project, on an eligible for QF chain
  • donors who are not "flagged as sybil for the round" (in our backend, set in admin bro)
  • Also we should have some setting in admin bro where we can mark if a round is completed or not... if the round is not complete, the unique donors should say "pending" cc @CarlosQ96

I move it back to inprogress to be checked @CarlosQ96 @mohammadranjbarz

@laurenluz
Copy link
Member Author

re: mid-page banner

in this issue #4171 I put the copy for the banner assuming it exists! but if we want to remove it, it's also ok with me. whatever is the least work 😅

re: "unique donors" number

@CarlosQ96 told me that this number is just the number of unique addresses that donated in the round. it is not considering passport score or flagged sybils or other information. I think, to not further block this issue... we can leave it as-is.

I'll create a new issue for making a new type of "unique donor" variable for each round in our backend, and once we get to building that, we can update this here... but for now, I think this doesn't need to be a blocker and we can just move forward with the simple "unique donors aka unique addresses who donated in the round" thing.

@laurenluz
Copy link
Member Author

@maryjaf if we exclude those edge cases/improvements, is this issue all ok then?

@maryjaf
Copy link
Collaborator

maryjaf commented Jun 5, 2024

  • 1-Should the mid page QF banner be shown in the page of archived QF? @laurenluz
  • @maryjaf if we exclude those edge cases/improvements, is this issue all ok then?

Yeah, all open problems have been fixed, but we have one question about "mid page QF banner "
Should the mid page QF banner be shown in the page of archived QF? @laurenluz

@Meriem-BM Meriem-BM self-assigned this Jun 6, 2024
@laurenluz
Copy link
Member Author

we can remove the mid-page banner

@jainkrati
Copy link
Collaborator

@Meriem-BM pls continue on this with guidance from @MohammadPCh

@Meriem-BM
Copy link
Member

Hey @maryjaf, the mid banner is removed now

@maryjaf
Copy link
Collaborator

maryjaf commented Jun 18, 2024

Hey @maryjaf, the mid banner is removed now

Thanks, this problem has been fixed

@divine-comedian divine-comedian moved this to Done in All-Devs Jun 26, 2024
@divine-comedian divine-comedian added the QF Quadratic Funding Related label Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Done P2 A normal to do item or feature. QF Quadratic Funding Related
Projects
Status: Done
Development

No branches or pull requests

10 participants