Skip to content

Handle org website being null when merging them through enrichment#1662

Merged
skwowet merged 6 commits intomainfrom
debug/org-enrich-merge
Oct 11, 2023
Merged

Handle org website being null when merging them through enrichment#1662
skwowet merged 6 commits intomainfrom
debug/org-enrich-merge

Conversation

@skwowet
Copy link
Copy Markdown
Collaborator

@skwowet skwowet commented Oct 11, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at 5df0293

Refactored organizationEnrichmentService.ts to skip domain lookup and service creation for organizations without websites. This reduces the load on the database and the memory usage.

🤖 Generated by Copilot at 5df0293

No findByDomain
If org.website is false
Winter of queries

Why

How

🤖 Generated by Copilot at 5df0293

  • Optimize organization enrichment service by skipping unnecessary calls and checks (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 added the Bug Created by Linear-GitHub Sync label Oct 11, 2023
@skwowet skwowet merged commit 6fca272 into main Oct 11, 2023
@skwowet skwowet deleted the debug/org-enrich-merge branch October 11, 2023 16:38
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.

1 participant