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

Add negative streak (#110) #113

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

Add negative streak (#110) #113

wants to merge 1 commit into from

Conversation

inad9300
Copy link

As noted in #110, a "negative streak" would be really helpful in knowing what you should be paying closer attention to every day. I have extended the existing code in the most straightforward way to achieve this.

Note that old streak values have not been recalculated. I think this is not too important because negative streaks don't show up in any of the histories, and it is rare that people go back many days manually just to see this kind of info (and even if they do, I don't think it's a big deal). That said, of course feel free to do something about this.

Please, someone test the code and verify that everything is behaving as expected.

@nbeloglazov
Copy link

Any update on this pull request? I'd love to have negative strikes in the app.

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