Skip to content

Conversation

themarolt
Copy link
Contributor

@themarolt themarolt commented Nov 7, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at 63fe321

Added a score column and a distinct function to improve data loading for members. Updated the IMemberData interface and the data.repo.ts file to reflect the changes.

🤖 Generated by Copilot at 63fe321

To match the schema and query
The score property was added, you see
To the IMemberData interface
And to load data with grace
A distinct function and a score column, nifty!

Why

How

🤖 Generated by Copilot at 63fe321

  • Add score property to IMemberData interface to store member's activity and reputation value (link)
  • Include score column in SQL query to fetch member data from database in data.repo.ts (link)
  • Remove duplicate member ids from array before querying database to improve performance and accuracy in data.repo.ts (link)

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.

@themarolt themarolt added the Bug Created by Linear-GitHub Sync label Nov 7, 2023
@themarolt themarolt merged commit 85dea3e into main Nov 8, 2023
@themarolt themarolt deleted the bugfix/add-score-to-members-C-2585 branch November 8, 2023 07:35
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