Skip to content
This repository has been archived by the owner on Aug 10, 2018. It is now read-only.

Make it work! #1

Merged
merged 13 commits into from
Feb 4, 2015
Merged

Make it work! #1

merged 13 commits into from
Feb 4, 2015

Conversation

cadecairos
Copy link
Contributor

Work in Progress, here so people can track things.

  • implement build system
  • build login ux
  • build navbar ux
  • build one-up view

@thisandagain
Copy link

😄

@alicoding
Copy link
Collaborator

  • When removing a make the frame should be reloaded


var Firehose = React.createClass({
getInitialState: function() {
return {
loggedIn: false,
isAdmin: false
isAdmin: false,
csrfToken: document.querySelector("meta[name='csrf-token']").getAttribute("content"),
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Move it to componentDidMount()

@cadecairos
Copy link
Contributor Author

When removing a make the frame should be reloaded

not sure how to trigger this - when I delete the last make, I get the "no makes" message.

@cadecairos cadecairos assigned alicoding and unassigned cadecairos Feb 3, 2015
@alicoding
Copy link
Collaborator

great patch! r+

@alicoding
Copy link
Collaborator

GOOD ONE DUDE! :D

cadecairos pushed a commit that referenced this pull request Feb 4, 2015
@cadecairos cadecairos merged commit 00e1ae6 into mozilla:master Feb 4, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants