-
Notifications
You must be signed in to change notification settings - Fork 22
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
Become next.js #42
Become next.js #42
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.
Some comments
import api from "../api"; | ||
import Loader from "../Loader"; | ||
//import configFactory from "../../config.js"; | ||
//const config = configFactory(); |
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.
Commented lines should be removed
} | ||
|
||
render() { | ||
var messages = this.state.messages.map((message, i) => { |
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.
Voi ei, legacyä!!
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.
Tulevan pullarin hommia refaktoroida.
</div> | ||
<div | ||
className="message__body" | ||
dangerouslySetInnerHTML={{ __html: message.body }} |
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.
Vakava tietoturva-aukko tässä 👀
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.
Tulevaan pullariin.
@@ -4,76 +4,38 @@ | |||
"description": "Koodiklinikka homepage", | |||
"author": "Riku Rouvila <riku.rouvila@leonidasoy.fi>", |
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.
Tän vois varmaan kanssa päivittää ;)
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.
Mihin?
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.
riku@okaycodes.fi tai @rikukissa voi varmaa päättää :)
Merging to get some momentum on this 🎉 |
There could be rough corners and such, but at least the page shows up and is identical to the current production page :)
There are some TODO items though; see
TODO.md
.