Skip to content

Upped thresholds for merge suggestions and included email and displayName to the computation#336

Merged
joanreyero merged 1 commit intomainfrom
hotfix/engagement-level-threshold
Dec 9, 2022
Merged

Upped thresholds for merge suggestions and included email and displayName to the computation#336
joanreyero merged 1 commit intomainfrom
hotfix/engagement-level-threshold

Conversation

@joanreyero
Copy link
Copy Markdown
Contributor

@joanreyero joanreyero commented Dec 9, 2022

Changes proposed ✍️

  • Upped the Levenshtein threshold for merge suggestions
  • Added displayName and email to the computation.

Checklist ✅

  • Label appropriately with Feature, Enhancement, or Bug.
  • Tests are passing.
  • New backend functionality has been unit-tested.
  • Environment variables have been updated:
    • Local frontend configuration: frontend/.env.dist.local, frontend/.env.dist.composed.
    • Local backend: backend/.env.dist.local, backend/.env.dist.composed.
    • Configuration docs have been updated.
    • Team members only: update environment variables in override, staging and production env. files and trigger update config script.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.
  • All changes have been tested in a staging site.
  • All changes are working locally running crowd.dev's Docker local environment.

@joanreyero joanreyero added the Improvement Created by Linear-GitHub Sync label Dec 9, 2022
@joanreyero joanreyero requested a review from epipav December 9, 2022 11:39
@joanreyero joanreyero merged commit 35c83ef into main Dec 9, 2022
@joanreyero joanreyero deleted the hotfix/engagement-level-threshold branch December 9, 2022 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Improvement Created by Linear-GitHub Sync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants