Skip to content

Conversation

skwowet
Copy link
Member

@skwowet skwowet commented Aug 21, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at 7927475

Implemented the auto-assignment feature for members and organizations. Added a new method findByDomain to the organizationRepository and organizationService classes to query organizations by email domain. Refactored the memberService class to use the new method and assign members to organizations accordingly.

🤖 Generated by Copilot at 7927475

Sing, O Muse, of the skillful code that was written
By the wise developers, who sought to enhance the app
With a feature that would link the members and the orgs
By the domains of their emails, as swift as Hermes' winged sandals.

Why

How

🤖 Generated by Copilot at 7927475

  • Add a feature that allows members to be automatically assigned to organizations based on their email domains (link, link, link)
  • Create a new static method findByDomain in organizationRepository.ts that queries the database for an organization with a matching website attribute (link)
  • Rename and use the findByDomain method in memberService.ts to find and assign the organization ID to the member data (link)
  • Add a new instance method findByDomain in organizationService.ts that delegates the query to the repository layer (link)
  • Refactor the declaration and initialization of the organizations array in memberService.ts to improve readability and structure (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.

@skwowet skwowet requested a review from garrrikkotua August 21, 2023 11:47
@skwowet skwowet self-assigned this Aug 21, 2023
@skwowet skwowet added the Bug Created by Linear-GitHub Sync label Aug 21, 2023
@skwowet skwowet merged commit e73d057 into main Aug 21, 2023
@skwowet skwowet deleted the bugfix/c-2026 branch August 21, 2023 14: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