Skip to content
This repository has been archived by the owner on Oct 10, 2024. It is now read-only.

Add /heatmap command #34

Merged
merged 21 commits into from
Jun 30, 2021
Merged

Add /heatmap command #34

merged 21 commits into from
Jun 30, 2021

Conversation

TimJentzsch
Copy link
Contributor

Relevant issue: Closes #29.

Description:

Adds a /heatmap command to display the activity heatmap of the given user.
The output looks like this:
Output of the /heatmap command
The image can be clicked for a larger view of the heatmap.

Please note that the current data is mostly incorrect because most dates in Blossom are still wrong (see GrafeasGroup/blossom#157).

Checklist:

  • Code Quality
  • Pep-8
  • Tests (if applicable)
  • Success Criteria Met
  • Inline Documentation
  • Wiki Documentation (if applicable)

@TimJentzsch TimJentzsch requested a review from a team as a code owner June 24, 2021 14:11
buttercup/cogs/heatmap.py Outdated Show resolved Hide resolved
buttercup/cogs/heatmap.py Outdated Show resolved Hide resolved
buttercup/cogs/heatmap.py Outdated Show resolved Hide resolved
@MaxVanDeursen
Copy link
Member

Great to see the activity in this repository, I feel like you're really making leaps with the functionality!

@TimJentzsch
Copy link
Contributor Author

Thanks @MaxVanDeursen for the suggestion, this really made the code way more concise!

I have already added two new features:

  • The username is now optional and the user executing the command is taken as default
  • The user can set their timezone in their display name and the heat map will update accordingly

Preview:
Response of the bot to the /heatmap command

buttercup/cogs/helpers.py Outdated Show resolved Hide resolved
buttercup/cogs/heatmap.py Outdated Show resolved Hide resolved
buttercup/cogs/heatmap.py Show resolved Hide resolved
@TimJentzsch TimJentzsch changed the title Add heatmap command Add /heatmap command Jun 29, 2021
This was referenced Jun 29, 2021
@MaxVanDeursen MaxVanDeursen merged commit 0f00d79 into master Jun 30, 2021
@MaxVanDeursen MaxVanDeursen deleted the 29-heatmap-command branch June 30, 2021 09:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add /heatmap command
2 participants