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

🐛 BUG: Streak Delayed/Untracked #1614

Open
1 task done
MCBrueggman opened this issue Nov 22, 2024 · 6 comments
Open
1 task done

🐛 BUG: Streak Delayed/Untracked #1614

MCBrueggman opened this issue Nov 22, 2024 · 6 comments

Comments

@MCBrueggman
Copy link

What went wrong? 🤔

Over the last 2 days, I tried to keep my streak up, however, it didn't track on Thursday the 21st, and said that I didn't read any articles that day, despite the fact that i made a post that day. and for today, I read an article and it tracked the article for yesterday

Expected Behavior

I should have a streak of 69 days, not 68 days, and it should show that I read plenty of articles on Thursday and Friday

Steps to Reproduce Issue

1. Go to landing page
2. Click on any article
3. Follow the link to said article
4. Read the article
5. Return to page to see streak increase

Solution Proposed

Just make sure that people's streaks don't fall on events like this

Screenshots

image image image

Environment

No response

Browsers

No response

OS

No response

Version of daily.dev

No response

Additional Context

No response

Code of Conduct

  • I follow the conditions of this project.
@sshanzel
Copy link
Member

Hi @MCBrueggman. Thank you for reporting. Can you check if you have your timezone correctly set through this page? https://app.daily.dev/account/notifications

@MCBrueggman
Copy link
Author

MCBrueggman commented Nov 22, 2024 via email

@sshanzel
Copy link
Member

Does the issue still persist after refresh? I checked in our DB and you seem to have your streak for today as recorded. However, the value 68 seems to be the right value for your account, and the issue is a UI glitch if it still does.

@MCBrueggman
Copy link
Author

MCBrueggman commented Nov 22, 2024 via email

@sshanzel
Copy link
Member

Yep. Raised a PR for it now. Once merged, it should fix the issue 👀

@vishxly

This comment was marked as spam.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants