-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add ability to set breathe in hold duration #2
base: main
Are you sure you want to change the base?
Conversation
7a549cb
to
06998e7
Compare
06998e7
to
88e3b99
Compare
src/app/helper.ts
Outdated
{duration: 30, breatheInHold: 15}, | ||
{duration: 60, breatheInHold: 15}, | ||
{duration: 90, breatheInHold: 15}], | ||
schemeLevel: 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The schemeLevel should be bumped and a migration should be done in getSettings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've added a migrateSettings function to migrate from v1 to v2 automatically, but not sure this is the cleanest way to do it?
onClick={() => this.removeRound(round)} | ||
/> | ||
</div> | ||
<Card variant="outlined" style={{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea - done.
104228c
to
f4bfae8
Compare
I also noticed the CICD is failing. I think I need to set something up for gh-pages to work properly? |
I wanted the ability to configure the breathe in hold to something other than 15sec, so I added it as a configurable setting with room to add others there later.
Not sure about the layout but it's workable.