-
Notifications
You must be signed in to change notification settings - Fork 1
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
#156882158 Landing page with effective UI/UX #91
Conversation
user can see error message on sign up page show image on sign up page [Delivers #157283077]
- Resign the landing page - Create css style for the landing page [Deliver #155882158]
public/js/controllers/index.js
Outdated
|
||
|
||
|
||
<<<<<<< HEAD |
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.
Parsing error: Unexpected token <<
Pull Request Test Coverage Report for Build 306
💛 - Coveralls |
8893b73
to
53e3f5d
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.
Great job implementing landing page.
@@ -8,3 +8,6 @@ coverage | |||
/data | |||
/.env | |||
/build | |||
privacy.html | |||
|
|||
meet-the-team.html |
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.
Great job. However, I think you should specify the path to the html files in case another file with the same name exists in another folder. Also, there should be a new line at the end of this file.
"clientSecret": "APP_SECRET", | ||
"callbackURL": "http://cfh.io/auth/facebook/callback" | ||
"clientID": "180680672583006", | ||
"clientSecret": "3cfd8dd9cb1116aedc701bd2138b66d2", |
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.
This is sensitive information. The secret token should not be included in the code. Use a dotenv file to hold this information instead.
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.
Good job with the redesign. But I discovered that some of your styles are not working well on Heroku. The Signup and login buttons on the landing page is not styled on Herkou. Please fix the style.
Here is the like to the deployed version. https://thorin-cfh-staging-pr-91.herokuapp.com. Thanks
- Resign the landing page - Create css style for the landing page [Deliver #155882158]
53e3f5d
to
6cb5602
Compare
What does this PR do?
Design landing page for the software product
Description of Task to be completed?
Users should see a landing page with effective UI/UX
How should this be manually tested?
What are the relevant pivotal tracker stories?
Story ID #156882158