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

Listen streak override step count #2389

Merged
merged 2 commits into from
Jan 26, 2022

Conversation

jowlee
Copy link
Contributor

@jowlee jowlee commented Jan 26, 2022

Description

Add override for step count
For listen streak - check if over 48 hours have passed since the last listen streak return 0 step count in override

Tests

Made a test for it

How will this change be monitored?

@jowlee jowlee added bug Something isn't working discovery-node Discovery Node (previously known as Discovery Provider) labels Jan 26, 2022
Copy link
Contributor

@piazzatron piazzatron left a comment

Choose a reason for hiding this comment

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

🛳

@jowlee jowlee force-pushed the jowlee-listen-streak-override-step-count branch from d696594 to 6baf948 Compare January 26, 2022 21:09
@jowlee jowlee merged commit e4b826a into master Jan 26, 2022
@jowlee jowlee deleted the jowlee-listen-streak-override-step-count branch January 26, 2022 21:24
raymondjacobson pushed a commit that referenced this pull request Jan 27, 2022
* Add override for step count listen streak

* Fix lint issues
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working discovery-node Discovery Node (previously known as Discovery Provider) size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants