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

pomodoro: Overhaul block #2020

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

pomodoro: Overhaul block #2020

wants to merge 1 commit into from

Conversation

bim9262
Copy link
Collaborator

@bim9262 bim9262 commented Mar 20, 2024

Changes:

The previous config will still be valid with the current config since there are new formats for the various states.

Ideally this will be merged after the format duration is added so that we don't have an additional inflection point in the config.

@bim9262 bim9262 linked an issue Mar 30, 2024 that may be closed by this pull request
@bim9262
Copy link
Collaborator Author

bim9262 commented Mar 31, 2024

With #2032 we won't need to expose two values for the current pomodoro number.

@bim9262 bim9262 force-pushed the pomodoro branch 2 times, most recently from 195523e to 5b592ac Compare March 31, 2024 01:25
@bim9262 bim9262 mentioned this pull request Jun 19, 2024
@aadilshabier
Copy link

Hey, is there any update on this PR? I wanted to the timers to the configuration, but saw that a PR was already in place.

@bim9262
Copy link
Collaborator Author

bim9262 commented Sep 5, 2024

Once #2032 is merged we can review this change @aadilshabier

@bim9262 bim9262 marked this pull request as ready for review September 7, 2024 20:59
src/blocks/pomodoro.rs Outdated Show resolved Hide resolved
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.

Pomodoro: Alternative number format: arabic / decimal
3 participants