gitcoinco / web

Grow Open Source
https://gitcoin.co
Other
1.78k stars 771 forks source link

GR11 Feedback #9422

Closed octavioamu closed 3 years ago

octavioamu commented 3 years ago

Internal feedback ticket

still working on fixes for:

willsputra commented 3 years ago

Thanks for this! A few feedback, other than the filter styles that you're working on:

abitrolly commented 3 years ago

https://gitcoin.co/hackathon/gr11/onboard the link to wiki page https://gitcoin.co/wiki/grants/ is 404.

image

abitrolly commented 3 years ago

And it would be nice to get these implemented before GR11 starts. As I've messed up a Twitter handle in my submission.

frankchen07 commented 3 years ago

Aditya: this has been fixed

frankchen07 commented 3 years ago

Aditya: this has been fixed

PixelantDesign commented 3 years ago

Lookin' good! Feels smooth and fast.

Aside from some of the items mentioned above...

octavioamu commented 3 years ago

~wrong filter condition probably when empty https://stage.gitcoin.co/grants/cards_info?page=1&limit=6&sort_option=weighted_shuffle&network=mainnet&state=active&collections_page=1&grant_types=&grant_tags=&tenants=&idle=true&sub_round_slug=gr11-uniswap&round_num=1&customer_name=Uniswap&tab=grants probably due a grant without any tag selected https://stage.gitcoin.co/_administrationgrants/grant/3/change/~

Aditya: The filter was not set right in DB. It should be {"tags": "1"}

octavioamu commented 3 years ago
octavioamu commented 3 years ago
octavioamu commented 3 years ago

not handling api server error messages

thelostone-mc commented 3 years ago
willsputra commented 3 years ago

When filtering term didn't match, there is no feedback on the page (this might still be wip still). Aditya: could you share design for this?

@thelostone-mc @octavioamu tagged you with the empty state designs on GR11 Figma!

willsputra commented 3 years ago

tried creating grants: https://stage.gitcoin.co/grants/11/grant1-with-gaming-wallet-tags https://stage.gitcoin.co/grants/12/grant2-with-daos-tag

octavioamu commented 3 years ago
willsputra commented 3 years ago

UI / styles tweaks (will keep updating this post for UI/styles stuff):

frankchen07 commented 3 years ago

Screen Shot 2021-09-02 at 09 13 10

Screen Shot 2021-09-02 at 12 00 13

the checkout buttons look funny, what's the random purple square?

Screen Shot 2021-09-02 at 12 02 17

owocki commented 3 years ago

all testing done on chrome/osx with metamask

https://bits.owocki.com/eDuRlw6B https://bits.owocki.com/rRubN09g

octavioamu commented 3 years ago

@willsputra how did you created this 2 grants? seems are without tenant that the reason on showing up on the cart since not eth grants and not indication of other tenant image

octavioamu commented 3 years ago
willsputra commented 3 years ago

@willsputra how did you created this 2 grants? seems are without tenant that the reason on showing up on the cart since not eth grants and not indication of other tenant image

hmm I chose ETH on the create grant form, although I think I put "0x0" on the recipient wallet address field

octavioamu commented 3 years ago

hmm I chose ETH on the create grant form, although I think I put "0x0" on the recipient wallet address field

frankchen07 commented 3 years ago

didn't we change the language and functionality to be a % on top of their original contribution?

Screen Shot 2021-09-02 at 12 03 20

PixelantDesign commented 3 years ago

Explorer List view

octavioamu commented 3 years ago

$0 out of alignment on card

that is due to the other bug on the form not having validation on address then not getting tenant, as there is not icon with the chain the template create that break

octavioamu commented 3 years ago

when on cart page if you decide to create a collection then you get redirected , this shouldn't happen as we don't even know if the user want to go to another place, maybe he just want to checkout next

octavioamu commented 3 years ago
PixelantDesign commented 3 years ago

when on cart page if you decide to create a collection then you get redirected , this shouldn't happen as we don't even know if the user want to go to another place, maybe he just want to checkout next

agree @octavioamu, lets keep the user in the checkout page after collections creation is complete.

PixelantDesign commented 3 years ago

from @owocki comment

why is there a random ether logo floating over here? https://bits.owocki.com/9ZuQD09Z what does this mean? theres no label, no descriptor, no alt text. what is it? its bugging me.

The intent was to denote what chain that grant accepted payment on. This is a confusion point for users as well. We could add a tooltip to clarify that. Thoughts @octavioamu ?

owocki commented 3 years ago

per the team call today, make sure we test the new endpoints with productionlike data pls. if a grant with 10k contributions takes several seconds to load, that ll be a problem when the code hits prod

Kweiss commented 3 years ago

Looks like most of the "sort by" are not working:

octavioamu commented 3 years ago
PixelantDesign commented 3 years ago
Kweiss commented 3 years ago
Lerog1724 commented 3 years ago

Good day, should I return the old address or I'll be sticking with the metamask?

Rogeljoanclitonkevin17242111$$$

On Fri, Sep 3, 2021, 9:44 PM Pixelant @.***> wrote:

from @owocki https://github.com/owocki comment

why is there a random ether logo floating over here? https://bits.owocki.com/9ZuQD09Z what does this mean? theres no label, no descriptor, no alt text. what is it? its bugging me.

The intent was to denote what chain that grant accepted payment on. This is a confusion point for users as well. We could add a tooltip to clarify that. Thoughts @octavioamu https://github.com/octavioamu ?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/gitcoinco/web/issues/9422#issuecomment-912551370, or unsubscribe https://github.com/notifications/unsubscribe-auth/AS73BNVENEXYMC5S74MW3WLUADGNHANCNFSM5DHD2EYQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

PixelantDesign commented 3 years ago

Cart dropdown - no link on each grant

Screen Shot 2021-09-07 at 11 20 47 AM

Checkout page - link exists for each grant

Screen Shot 2021-09-07 at 11 20 36 AM

Aditya: Let's have this as a new ticket.

PixelantDesign commented 3 years ago
octavioamu commented 3 years ago

Guys if you have something not related to the changes we did on the sprint and is not a blocker please create a separate ticket.

willsputra commented 3 years ago

testing on prod:

Screenshot 2021-09-08 at 8 17 55 PM
PixelantDesign commented 3 years ago
PixelantDesign commented 3 years ago

Brave Screenshot 2021-09-09 at 12 35 26 AM

octavioamu commented 3 years ago

Closing for bug reports create a new issue or use internal CS-L2