Skip to content
This repository has been archived by the owner on Sep 22, 2020. It is now read-only.

Node update #395

Merged
merged 3 commits into from
May 16, 2018
Merged

Node update #395

merged 3 commits into from
May 16, 2018

Conversation

gideonthomas
Copy link
Contributor

Will update description when i get a sec. One major change I made was I removed js beautifier because it was breaking everything. I thought we don't really need it but let me know if we do.

@gideonthomas
Copy link
Contributor Author

ugh, I'll fix the travis errors

@cadecairos
Copy link
Contributor

@gideonthomas poke

@gideonthomas
Copy link
Contributor Author

@cadecairos sorry, I'll try to get to this sometime this week. Until then would you be able to take one final look at the id patch: mozilla/id.webmaker.org#447

@cadecairos
Copy link
Contributor

got any time to invest on this in the next little while @gideonthomas?

@gideonthomas
Copy link
Contributor Author

yeah, the creator patch for pulse was the biggest issue that needed to be fixed. I have one small follow-up to do and then I can revisit this.

Copy link
Contributor

@cadecairos cadecairos left a comment

Choose a reason for hiding this comment

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

This seems to work just fine locally, lets get it on staging and kick the tires.

@cadecairos cadecairos merged commit f2cc92e into master May 16, 2018
@cadecairos cadecairos deleted the node-update branch May 16, 2018 15:10
@cadecairos cadecairos restored the node-update branch May 16, 2018 15:51
@cadecairos cadecairos mentioned this pull request May 16, 2018
@cadecairos cadecairos mentioned this pull request Jun 18, 2018
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.

2 participants