Skip to content

Conversation

skwowet
Copy link
Member

@skwowet skwowet commented Sep 12, 2023

Changes proposed ✍️

What

🤖 Generated by Copilot at 65b4a16

This pull request adds two database migrations to normalize the website column of the organizations table and create a new unique index on it. It also adds two new methods to the dataSink.repo.ts file to query segment ids and member ids and emails for a given tenant and segment.

🤖 Generated by Copilot at 65b4a16

website index dropped
normalizing values with script
fall leaves no rollback

Why

How

🤖 Generated by Copilot at 65b4a16

  • Normalize website values for organizations and add unique index (link, link)
  • Add new methods to dataSink.repo.ts for querying segment ids and member ids and emails (link, 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 Improvement Created by Linear-GitHub Sync label Sep 12, 2023
@skwowet skwowet merged commit 081005b into main Sep 22, 2023
@skwowet skwowet deleted the database/normalize-website branch September 22, 2023 07:47
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.

2 participants