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

KS Grants QA - 11.16.18 #2862

Closed
39 of 51 tasks
captnseagraves opened this issue Nov 16, 2018 · 5 comments
Closed
39 of 51 tasks

KS Grants QA - 11.16.18 #2862

captnseagraves opened this issue Nov 16, 2018 · 5 comments
Assignees
Labels
Gitcoin Grants Gitcoin Grants

Comments

@captnseagraves
Copy link
Contributor

captnseagraves commented Nov 16, 2018

Index

  • Implement index revamp design
  • Need a link to ‘Your Grants’ in nav or in user dropdown
  • Need paginator instead of infinite scroll
  • Change ETH in Card progress bar to be DAI
  • Short images shorten card length. We should enforce a certain height to maintain consistency.

New

  • It would be nice to spice up design
  • If goal value in DAI is 0 all views break because progress bar divides by 0, which is NaN. Need validation to ensure amount goal is greater than 0.
  • Rename “title” input tag so that google doesn’t try to guess with your name
    screen shot 2018-11-16 at 2 34 49 pm
  • "create Grants button should be same width as form and same color as other action buttons. See "fund Grant" button for example
  • When the ‘waiting_room’ screen is up, the ALPHA warning banner still sitting at the bottom of the page
  • remove ETH from token drop down and include "any"

What is Grants

  • Introduce yourself or your team bullet appears twice
  • “Contribute with any ERC-20 token’ bullet says that you can pay with ETH, but you can’t

Details/Admin

  • Title input cannot handle a long title. Cuts title off after ~15-20 characters.
  • Same for url, but a little longer since the link column is wider
  • Need to change ETH in progreess bar to DAI
  • Should users be able to edit the their monthly goal?
  • Need to wire up updates
  • POST on updates fire the cancel_grant functionality on frontend. Will likely have the same issue when grant edit info is saved. I have a solution, but need to implement.
  • ‘Waiting_room’ scene need to be implemented for cancel_grant functionality
  • Metamask check need to be implemented for cancel_grant funcitonality
  • Need to smooth out design when cancel_grant fired and “This grant has ended. You can find all active grants here.” is displayed.
  • After a user cancels their subscription, grants/deatils should no loner show them as a supporter

Milestones

  • Would be nice to spice up design
  • Should we suggest people create milestones in the Grant Creation Process?
  • Date entered should be the deadline completion data, not the data the milestone is started.
  • The complete button should mark the milestone as completed and the actual date the milestone is marked completed with an associated github commit hash.

Your Grants

  • Need paginator on both supported and created tabs
  • Need to implement transactions on both tabs
  • Need link to “your grants” either in nav or in user dropdown. Right now no way to access it.

Details/User

  • Can we make the “Cancel Your Funding” button red? Or something other than the same blue as “Fund This Grant”.
  • Would be nice if users could see how much they are contributing and in what token.

Fund

  • Grant logo is super huge after update to datails
  • Would be nice to revamp design
  • Need to change ETH to DAI in progress bar
  • Could we abstract token approval limit to “Number of Payments”? The we could just multiply the payment value by the number of payments.
  • Metamask check works, but has grant logo in the middle of it right now.
  • Need to check whether funer is admin and disallow contribution
  • need to check whether grant is active or not and allow functionality accordingly
  • make sure a user can only create one subscription per grant in backend. currently only stopping this by selectively showing "Fund Grant" button

Cancel_subscription

  • Metamask check works, but has grant logo in the middle of it
  • Cancel_subscription seems to be broken when subscription values aren’t 0. Thought i fixed this, but it might have been merged over.

Larger build pieces

  • Integrate subminer
  • Make sure transaction data flows correctly to all progress bars, transaction history in Profile, and Emails
  • QA Emails
  • Implement Update POST Functionality
  • Implement Grant Info POST functionality
  • Cancel Subscription is broken
  • We need to make sure that a user is not able to choose different variables in their subscription if the grants creator has set requirements in their contract. Mostly pertains to tokens.

Outstanding Questions

  • What happens when a user tries to create a new grant without a profile? Where are they caught? Do we have an error screen for this?
  • Is there any check for a users wallet? I can contribute to a grant and then go to try and cancel it with a different address. Which will fail. Can we validate that at all?
@thelostone-mc
Copy link
Member

@captnseagraves title & link input both are set to width 100% but they are still input fields which prevents them from being broken up into multiple lines.

This is only when you the logged in user owns the grant. Any suggestions on how we go about this ?

thelostone-mc added a commit to thelostone-mc/web that referenced this issue Nov 19, 2018
- remove duplicate bullet + ETH in how it works
- revert to old image size of grant/fund page
- details -> made title input 100% width
- alpha message now appears before waiting state
- cancel your funding is not red

Refs: gitcoinco#2862
This was referenced Nov 19, 2018
@captnseagraves
Copy link
Contributor Author

@thelostone-mc Sorry I didnt respond to this yesterday. I implemented a solution that has already been merged. I used a textarea for each to resolve the issue.

@captnseagraves
Copy link
Contributor Author

KO QA #2897

@mbeacom
Copy link
Contributor

mbeacom commented Nov 20, 2018

  • What happens when a user tries to create a new grant without a profile? Where are they caught? Do we have an error screen for this?

We require the user to be authenticated in order to create a grant. They are redirected to auth automatically.

@thelostone-mc
Copy link
Member

Open issues are captured here #2949

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Gitcoin Grants Gitcoin Grants
Projects
None yet
Development

No branches or pull requests

4 participants