Skip to content

Sort merge suggestions by confidence#1826

Merged
joanreyero merged 2 commits intomainfrom
improvement/merge-suggestions-sorted-similarity
Nov 7, 2023
Merged

Sort merge suggestions by confidence#1826
joanreyero merged 2 commits intomainfrom
improvement/merge-suggestions-sorted-similarity

Conversation

@joanreyero
Copy link
Copy Markdown
Contributor

@joanreyero joanreyero commented Nov 7, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at 0503354

Improved the member merge feature by sorting the candidates by similarity score. Modified the getMembersToMerge method in memberRepository.ts to implement the new ordering logic.

🤖 Generated by Copilot at 0503354

getMembersToMerge
Sorts by similarity score
Autumn leaves falling

Why

How

🤖 Generated by Copilot at 0503354

  • Order the results of getMembersToMerge by similarity score instead of joined date (link). This improves the user experience by showing the most similar members first in the merge interface. The similarity score is calculated using a combination of factors such as name, email, location, and skills in the backend/src/database/repositories/memberRepository.ts file.

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screenshots to the PR description for relevant FE changes
  • New backend functionality has been unit-tested.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.

@joanreyero joanreyero added the Improvement Created by Linear-GitHub Sync label Nov 7, 2023
@joanreyero joanreyero changed the title Changed Sort merge suggestions by confidence Nov 7, 2023
@joanreyero joanreyero merged commit 376c625 into main Nov 7, 2023
@joanreyero joanreyero deleted the improvement/merge-suggestions-sorted-similarity branch November 7, 2023 14:10
sausage-todd pushed a commit that referenced this pull request Nov 9, 2023
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.

1 participant