Add first_active field to BlossomUser model #147
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Relevant issue: Closes #141.
Description:
The Slack
info
command used thedate_joined
field ofBlossomUser
. However, this is actually never assigned anywhere, so it always returned the current time.This adds a separate
first_active
field which returns the time of the first claimed transcription. This is then used instead of thedate_joined
field for theinfo
command.Testing Instructions:
Locally setup the Blossom Testing App. Then you can test the
info <username>
command and verify that thedate_joined
field has been replaced byfirst_active
and returns a sensible value.Because I didn't manage to set up Slack locally yet, it would be nice if someone could verify this for me.
Checklist: