Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: cleanup selectors #10078

Merged
merged 24 commits into from
Aug 6, 2024
Merged

chore: cleanup selectors #10078

merged 24 commits into from
Aug 6, 2024

Conversation

mattkrick
Copy link
Member

@mattkrick mattkrick commented Aug 5, 2024

Description

i created a few bad patterns while migrating to PG & this undoes them so no one copies the bad patterns.

we also had a bad pattern where we used a global GraphQLID throughout the server, including dataloaders, and even database fetches. we even stored that ID in the database & sent it off via analytics. I gotta get rid of that so we don't do it again because it makes figuring out which ID to use where really hard.

jordanh and others added 20 commits March 3, 2024 09:55
Signed-off-by: Matt Krick <[email protected]>
Signed-off-by: Matt Krick <[email protected]>
Signed-off-by: Matt Krick <[email protected]>
Signed-off-by: Matt Krick <[email protected]>
Signed-off-by: Matt Krick <[email protected]>
Signed-off-by: Matt Krick <[email protected]>
@github-actions github-actions bot requested a review from tianrunhe August 5, 2024 23:17
@github-actions github-actions bot added size/m and removed size/xl labels Aug 5, 2024
Signed-off-by: Matt Krick <[email protected]>
Signed-off-by: Matt Krick <[email protected]>
@github-actions github-actions bot added size/l and removed size/m labels Aug 6, 2024
@mattkrick mattkrick merged commit 0a6534d into master Aug 6, 2024
7 checks passed
@mattkrick mattkrick deleted the chore/cleanup-selectors branch August 6, 2024 19:48
@github-actions github-actions bot mentioned this pull request Aug 8, 2024
24 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants