Skip to content

[DO NOT MERGE WIP] jyfan/strict-alarm#1832

Open
jykfan wants to merge 1 commit intomasterfrom
jyfan/insight-strict-alarm
Open

[DO NOT MERGE WIP] jyfan/strict-alarm#1832
jykfan wants to merge 1 commit intomasterfrom
jyfan/insight-strict-alarm

Conversation

@jykfan
Copy link
Contributor

@jykfan jykfan commented Nov 8, 2016

In this PR:

  • Asks user for consent to set a single alarm clock
  • Current condition: User wake variance is above some threshold. User has enough sleep stats during the week to figure out an average wake up time

To change:

  • Change condition to: User has different alarms set for weekdays and weekend. User has enough sleep stats during the week to figure out an average wake up time.
  • Test questions UI

More implementation:

  • Testing on admin insight endpoint
  • Immediately set alarm in response endpoint
  • Scheduling "insight" under feature flipper

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.

1 participant