Skip to content

Conversation

@nuisagumen
Copy link

Changed the am/pm logic as requested by Beth: starts at pm instead of am, removed 0 am as it does not exist.

Copy link
Contributor

@pmkihara pmkihara left a comment

Choose a reason for hiding this comment

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

LGTM!

I think we should wait for USC reply about the time dropdown before we merge this. We might want to add another parameter to check whether to display the time in the normal order or with pm before am if they want to keep the normal order for the wake time question.

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