Skip to content
This repository was archived by the owner on Mar 17, 2025. It is now read-only.

Neopixel demo #64

Merged
merged 3 commits into from
Apr 8, 2016
Merged

Neopixel demo #64

merged 3 commits into from
Apr 8, 2016

Conversation

gguuss
Copy link
Contributor

@gguuss gguuss commented Feb 4, 2016

A basic demo of Web + Arduino control of an Adafruit Neopixel feather shield


@ed7coyne
Copy link
Collaborator

looks pretty slick, I like the web preview.



// TODO: Replace with your own credentials and keep these safe.
Firebase fbase = Firebase("brilliant-fire-9638.firebaseio.com")
Copy link
Contributor

Choose a reason for hiding this comment

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

you might want to remove your secret from the sample :)

Firebase fbase = Firebase("brilliant-fire-9638.firebaseio.com")
.auth("llXzJtqwdiHCHw6F6sehQR2EPwbO0YYdhEjPRZxG");
Firebase fbase = Firebase("YOUR-PROJECT-ID.firebaseio.com")
.auth("YOUR_AUTH_SECRET");
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe use the same placeholder that we use in the other sample:
https://github.com/googlesamples/firebase-arduino/blob/master/examples/FirebasePush_ESP8266/FirebasePush_ESP8266.ino#L23

Or change the other sample to use yours :) (I'm not sure Project ID is in firebase jargon)

@proppy
Copy link
Contributor

proppy commented Mar 24, 2016

Maybe you can rename the -web subfolder to just www or web and add intruction to deploy the site with https://www.firebase.com/docs/hosting/quickstart.html

Maybe also add a README with the same description than this pull request.

@proppy
Copy link
Contributor

proppy commented Mar 24, 2016

Just some nits, but LGTM.

With the dependencies set, serve the www folder from a web server. For example,
you can use the Python Simple HTTP server module by running:

`python -m SimpleHTTPServer [port]`
Copy link
Contributor

Choose a reason for hiding this comment

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

@proppy
Copy link
Contributor

proppy commented Apr 8, 2016

LGTM

@proppy proppy merged commit 7bad03f into FirebaseExtended:master Apr 8, 2016
@gguuss gguuss deleted the neopixel branch November 17, 2016 03:22
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.

3 participants