-
Notifications
You must be signed in to change notification settings - Fork 161
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
feat: add all time view option for analytics #3009
Conversation
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.
Granularity needs to change to month
c51d525
to
6140d55
Compare
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.
Also, please don't merge and only rebase. I took a few minutes to clean up the commits on this PR to be more inline with our syntax. |
6140d55
to
d4812b6
Compare
Sure. Yes, this PR is not ready to merge. I initially thought I needed to change the backend, so I was looking into the server codebase. |
@skeptrunedev, will there be any start date for |
6c9a105
to
8139a31
Compare
"all time" working properly, now doing the backend work to implement month granularity |
3fbcad0
to
8f92730
Compare
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.
Closing PR as we are running into issues grouping by month using the clickhouse rust crate. Reopening as draft so that @densumesh can take a look at whats going on with rust crate before we proceed. Thank you @ptdatta for your work! |
8f92730
to
0dc58a1
Compare
Screen.Recording.2025-01-01.at.3.11.26.PM.mov |
Please indicate what issue this PR is related to and @ any maintainers who are relevant
Fixes #2986