-
Notifications
You must be signed in to change notification settings - Fork 18
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
Removing unused activity components #774
Conversation
Could you update the PR title to something more descriptive before merging? It makes release notes and the commit history easier to follow :) |
0dd61e8
Thanks for catching that. Not sure how the initial title even came to be. 😅 |
Could we provide the context for removing |
Good call, I have linked the related issue to it. |
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.
Nicely done!
Relates to:
#720
#775
Summary
Removes the code for individual activity APIs as the frontend uses the consolidated
/metric
endpoint as of #749Also, removes
totalMonths
in the metric endpoint response as a part of #775