Skip to content

Conversation

themarolt
Copy link
Contributor

@themarolt themarolt commented Jul 4, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at e69de53

Simplified SQL query in member.repo.ts to find members by platform and username. This improves readability and performance of the data sink worker service.

🤖 Generated by Copilot at e69de53

findMember query
Simplified with inner join
Spring cleaning the code

Why

How

🤖 Generated by Copilot at e69de53

  • Simplify SQL query to find member by platform and username (link)

Checklist ✅

  • Label appropriately with Feature, Improvement, or Bug.
  • Add screehshots 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.

@themarolt themarolt requested a review from garrrikkotua July 4, 2023 13:03
@themarolt themarolt added the Bug Created by Linear-GitHub Sync label Jul 4, 2023
@themarolt themarolt merged commit 58fdf5b into main Jul 4, 2023
@themarolt themarolt deleted the bugfix/optimize-find-member-query branch July 4, 2023 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Created by Linear-GitHub Sync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants