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

Transaction Support #156

Merged
merged 39 commits into from
Sep 18, 2021
Merged

Transaction Support #156

merged 39 commits into from
Sep 18, 2021

Conversation

tyt2y3
Copy link
Member

@tyt2y3 tyt2y3 commented Sep 14, 2021

#142 continue

@tyt2y3
Copy link
Member Author

tyt2y3 commented Sep 14, 2021

Let's get movin @billy1624

@nicoulaj
Copy link

Maybe you could include ConnectionTrait in the prelude ? Btw, QuerySelect and QueryOrderare also missing (QueryFilter, QueryResult are included so it feels a bit inconsistent).

@tyt2y3
Copy link
Member Author

tyt2y3 commented Sep 18, 2021

@nappa85 well, I tried to rebase the branch onto master, which is quite daunting indeed. Let me know if anything is missing.

@nappa85
Copy link
Contributor

nappa85 commented Sep 18, 2021 via email

@tyt2y3 tyt2y3 changed the base branch from master to transaction-1 September 18, 2021 17:35
@tyt2y3 tyt2y3 merged commit 242d16d into transaction-1 Sep 18, 2021
@tyt2y3 tyt2y3 deleted the transaction branch September 18, 2021 17:35
@tyt2y3 tyt2y3 mentioned this pull request Sep 18, 2021
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants