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
Homepage #5
Homepage #5
Conversation
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.
Looking good overall, some smaller questions and some lorem ipsum remaining but otherwise good :)
package.json
Outdated
@@ -2,7 +2,7 @@ | |||
"name": "web", | |||
"version": "0.1.0", | |||
"private": true, | |||
"homepage": "https://www.elixirbench.org", | |||
"homepage": "https://elixir-bench.gigalixirapp.com", |
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.
? Shouldn't this stay as is because that's where we'll have it deployed and that's where it should go?
public/CNAME
Outdated
@@ -1 +0,0 @@ | |||
www.elixirbench.org |
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 need this to keep the URL/DNS record intact
|
||
const IndexPage = ({ classes }) => ( | ||
<Page> | ||
|
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.
weird blank line nothing big though :D
<Typography align='justify' className={ classes.gridDescription }> | ||
ElixirBench is a platform for running benchmarks for projects hosted | ||
on Github.Etiam hendrerit mattis sapien, eu placerat eros posuere | ||
vitae. Duis accumsan porta nunc et scelerisque. Lorem ipsum dolor sit amet. |
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.
might wanna get rid off the lorem ipsum :)
Lorem ipsum dolor sit amet, consectetur adipiscing elit. Integer venenatis | ||
diam id quam lobortis consequat. Lorem ipsum dolor sit amet, consectetur | ||
adipiscing elit. Etiam hendrerit mattis sapien, eu placerat eros posuere | ||
vitae. Duis accumsan porta nunc et scelerisque. |
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.
?
oisuer Lorem ipsum dolor sit amet, consectetur adipiscing elit. Integer venenatis | ||
diam id quam lobortis consequat. Lorem ipsum dolor sit amet, consectetur | ||
adipiscing elit. Etiam hendrerit mattis sapien, eu placerat eros posuere | ||
vitae. |
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.
?
Signed-off-by: Tallys Martins <tallysmartins@gmail.com>
No description provided.