-
-
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
Quickstart PR with corrected redirect #1308
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1308 +/- ##
==========================================
+ Coverage 30.63% 30.64% +0.01%
==========================================
Files 124 124
Lines 8784 8790 +6
Branches 1141 1141
==========================================
+ Hits 2691 2694 +3
- Misses 5985 5988 +3
Partials 108 108
Continue to review full report at Codecov.
|
…into feature/quickstart-screen
This is currently live on https://stage.gitcoin.co Bugs so far:
|
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.
We'll want to address the bugs in my first comment before merging this.
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
app/assets/v2/css/base.css
Outdated
background-color: #FF6700; | ||
} | ||
|
||
|
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.
extra line :P
app/assets/v2/css/base.css
Outdated
} | ||
|
||
.submit-notification a { | ||
color: #fff; |
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.
Don't hate me for this but can we uppercase this ( to keep it uniform )
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.
or lower case everything! we'll have work either way
LGTM left a few minor comments |
9d1f94f
to
0c25ca4
Compare
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
Checklist
Affected core subsystem(s)
Testing
Refers/Fixes
Refs #1029