Skip to content

[EdgeDB] Create Partnership queries #3122

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
May 7, 2024
Merged

[EdgeDB] Create Partnership queries #3122

merged 2 commits into from
May 7, 2024

Conversation

willdch
Copy link
Contributor

@willdch willdch commented Mar 19, 2024

https://seed-company-squad.monday.com/boards/5989610236/pulses/5792970379

@CarsonF Still work to be done but would love to get your eyes on these changes while working on the rest.

@willdch willdch requested a review from bryanjnelson March 19, 2024 22:28
@willdch willdch self-assigned this Mar 19, 2024
@willdch willdch requested a review from CarsonF as a code owner March 19, 2024 22:28
CarsonF
CarsonF reviewed Mar 20, 2024
View reviewed changes
Copy link
Member

@CarsonF CarsonF left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like you are on the right track!

@bryanjnelson bryanjnelson changed the title Create Partnership queries [EdgeDB] Create Partnership queries | refactor service/repo layers appropriately Mar 20, 2024
@willdch willdch force-pushed the 0937-edgedb-partnership-queries branch 2 times, most recently from 720a8bf to f1aef57 Compare April 2, 2024 21:38
@willdch willdch marked this pull request as draft April 2, 2024 21:39
@willdch willdch force-pushed the 0937-edgedb-partnership-queries branch from f1aef57 to 6929eac Compare April 16, 2024 17:31
@willdch willdch marked this pull request as ready for review April 17, 2024 20:57
bryanjnelson
bryanjnelson previously requested changes Apr 18, 2024
View reviewed changes
Copy link
Contributor

@bryanjnelson bryanjnelson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good!

@willdch willdch requested a review from bryanjnelson April 18, 2024 22:37
@willdch willdch force-pushed the 0937-edgedb-partnership-queries branch 2 times, most recently from e53ae4c to 4edddc3 Compare April 22, 2024 17:42
CarsonF
CarsonF reviewed Apr 24, 2024
View reviewed changes
@willdch willdch force-pushed the 0937-edgedb-partnership-queries branch from 4edddc3 to 1c551b8 Compare April 26, 2024 18:41
CarsonF
CarsonF reviewed Apr 30, 2024
View reviewed changes
CarsonF
CarsonF reviewed Apr 30, 2024
View reviewed changes
@willdch willdch requested a review from CarsonF May 7, 2024 15:21
@CarsonF CarsonF force-pushed the 0937-edgedb-partnership-queries branch from 838cca1 to 5ce7afe Compare May 7, 2024 15:54
@CarsonF CarsonF changed the title [EdgeDB] Create Partnership queries | refactor service/repo layers appropriately [EdgeDB] Create Partnership queries May 7, 2024
@CarsonF CarsonF enabled auto-merge (squash) May 7, 2024 15:56
CarsonF
CarsonF approved these changes May 7, 2024
View reviewed changes
Copy link
Member

@CarsonF CarsonF left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI @willdch checkout 5ce7afe

@CarsonF CarsonF merged commit 882f469 into develop May 7, 2024
21 checks passed
@CarsonF CarsonF deleted the 0937-edgedb-partnership-queries branch May 7, 2024 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CarsonF CarsonF CarsonF approved these changes

@bryanjnelson bryanjnelson Awaiting requested review from bryanjnelson

Assignees

@willdch willdch

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@willdch @CarsonF @bryanjnelson