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

Simplify Hue for new users #40

Merged
merged 1 commit into from
Jun 28, 2015
Merged

Simplify Hue for new users #40

merged 1 commit into from
Jun 28, 2015

Conversation

nfarina
Copy link
Contributor

@nfarina nfarina commented Jun 28, 2015

  • Sample config auto-discoveres Bridge IP
  • Username left blank so that the server will prompt you
  • Hue platform walks you through username creation (in as friendly a
    way currently possible)

/cc @stephenyeargin, #28

- Sample config auto-discoveres Bridge IP
- Username left blank so that the server will prompt you
- Hue platform walks you through username creation (in as friendly a
way currently possible)
@stephenyeargin
Copy link
Contributor

👍 I think you just cleared it off @madmod's todo list 😄

@madmod
Copy link
Contributor

madmod commented Jun 28, 2015

👍 Looks good. I can't try it out right now, but I will test it in a few hours. I think if we could add some ability for saving to the config.json we wouldn't need to stop running if they don't have a username yet.

@nfarina
Copy link
Contributor Author

nfarina commented Jun 28, 2015

Yes agreed it would be great to have either a method for updating config.json (which I'm not super into, it feels wrong to fiddle with "your" config) or a method for accessory storage of preference data. Or an official method of "first time setup" for accessories. Lots to think about...

nfarina added a commit that referenced this pull request Jun 28, 2015
@nfarina nfarina merged commit 6900113 into master Jun 28, 2015
@nfarina nfarina deleted the hue-create-username branch June 28, 2015 20:31
@maddox
Copy link
Contributor

maddox commented Jun 28, 2015

👎 on writing to the config
👍 to some sort of local storage (if more than 2 or 3 cases can be come up with)
👍 to not even thinking about it on this pull 😄

@maddox
Copy link
Contributor

maddox commented Jun 28, 2015

Ps: this pull is cool. Love it.

@nfarina
Copy link
Contributor Author

nfarina commented Jun 28, 2015

Thanks but it's just icing on the cake - thanks to @stephenyeargin and @madmod for doing all the real work supporting Hue! This was the last thing in my home that wasn't hooked up to HomeKit 😸

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.

None yet

4 participants