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

Add overwrite_check_percentage field to BlossomUser #304

Merged
merged 7 commits into from
Jan 7, 2022

Conversation

TimJentzsch
Copy link
Contributor

@TimJentzsch TimJentzsch commented Jan 6, 2022

Relevant issue: Closes #303

Description:

This adds the functionality to watch a user, overwriting the percentage used to determine if a transcription needs to be checked by a mod.
This has been requested by the user and welcome teams.

To accomplish this, an overwrite_check_percentage field has been added to the BlossomUser model, defaulting to None (automatic checks).

The mods can use the @Blossom watch <username> <percentage> command to overwrite this.
For example, with @Blossom Tim3303 75, transcriptions by Tim3303 will have a 75% to be checked by the mods.
The <percentage> option can be omitted and defaults to 100.

The check message in Slack will reflect if the percentage was overwritten, to make it easier to keep track of this.

Using the @Blossom unwatch <username> command, this behavior will be reset to the automatic algorithm.

Checklist:

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

@TimJentzsch TimJentzsch marked this pull request as ready for review January 7, 2022 15:01
@TimJentzsch TimJentzsch requested a review from a team as a code owner January 7, 2022 15:01
Copy link
Member

@itsthejoker itsthejoker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Damn dude, this looks fantastic! Give it a try!

@TimJentzsch TimJentzsch merged commit a7a6c9a into master Jan 7, 2022
@TimJentzsch TimJentzsch deleted the 303-override-check-percentage branch January 7, 2022 15:11
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.

RFC: Add check percentage override field
2 participants