-
-
Notifications
You must be signed in to change notification settings - Fork 775
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
qa+ corner cases #2997
qa+ corner cases #2997
Conversation
- fixed gitcoin_bot shrug + added grants white svg - added grant white svg icon to landing page - redirects to error page when user tries to fund / cancel subscription on inactive grant - redirects to error page when user tried to cancel already cancelled subscription
- updated youtube link + text on index - update quickstart info - fix nav dropdown on hover on stub page Refs: gitcoinco#2949
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor nit on the if
checks. Generally, lgtm.
🍰 🙌
Codecov Report
@@ Coverage Diff @@
## grantz #2997 +/- ##
==========================================
+ Coverage 30.03% 30.26% +0.23%
==========================================
Files 190 190
Lines 14490 14219 -271
Branches 1931 1870 -61
==========================================
- Hits 4352 4304 -48
+ Misses 9994 9777 -217
+ Partials 144 138 -6
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
subscription on inactive grant
cancelled subscription
QA + fixes
Refers/Fixes