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 trophies to activity records, in slack messages #57

Merged
merged 7 commits into from
Mar 2, 2024
Merged

Conversation

caarmen
Copy link
Owner

@caarmen caarmen commented Feb 11, 2024

Issue:

Changes:

  • Add repository function to retrieve top stats for a user and activity type
  • Update the fitbit process_new_activity usecase to use this new function and provide the info to the slack usecase_post_activity.
  • Update the slack usecase_post_activity to include the medals in the message.
  • Add tests

@caarmen caarmen force-pushed the top-activities branch 6 times, most recently from 5f4d377 to bfb7379 Compare February 13, 2024 22:07
@caarmen caarmen force-pushed the top-activities branch 2 times, most recently from c38bc57 to a5f49b1 Compare February 14, 2024 21:33
@caarmen caarmen force-pushed the top-activities branch 2 times, most recently from a3eeceb to a5a2aee Compare February 14, 2024 21:48
@caarmen caarmen changed the title [WIP 🚧 ] Add medals to activity records, in slack messages Add medals to activity records, in slack messages Mar 2, 2024
@caarmen caarmen marked this pull request as ready for review March 2, 2024 21:12
@caarmen caarmen merged commit 74960ad into main Mar 2, 2024
2 checks passed
@caarmen caarmen deleted the top-activities branch March 2, 2024 21:12
@caarmen caarmen changed the title Add medals to activity records, in slack messages Add trophies to activity records, in slack messages Mar 2, 2024
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.

1 participant