Skip to content
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

Fixing issue #24 on edfungus repository #25

Merged
merged 6 commits into from
Mar 25, 2017
Merged

Conversation

wickwire
Copy link
Contributor

@wickwire wickwire commented Oct 6, 2016

#24

Solved in the bug description, tried updating some of the package files on my fork to ease things up, may require more work in order to fully garantee Polymer 1.7.0 compatibility.

Updating from edfungus/Crouton
… account of polymer 1.7.0 changes. Also generated updated package.json and bower.json files
…ng the connections between clients (python demo + Crouton dashboard) and the CloudMQTT Broker.
…nd programming an actual ESP8266-12. 3 toggle switches and 2 youtube streams (same source for both)
@edfungus
Copy link
Owner

Give me some time to look at it and merge it

@wickwire
Copy link
Contributor Author

Sure, and perhaps I've been too eager to make the pull request - perhaps this needs more attention on upgrading to Polymer 1.7.0 - my changes may still allow for Polymer version mismatches during npm install, so maybe some more effort should be needed before cleanly fixing this.

@edfungus edfungus merged commit 7da3562 into edfungus:master Mar 25, 2017
@edfungus
Copy link
Owner

Long time over due -_- sorry

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants