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

[cloud related] migration/beanie to sqlalchemy #1811

Merged
merged 16 commits into from
Jul 3, 2024

Conversation

aybruhm
Copy link
Member

@aybruhm aybruhm commented Jun 24, 2024

Description

This PR migrates beanie related code for cloud/ee to sqlalchemy-orm.

Copy link

vercel bot commented Jun 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
agenta ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 1, 2024 5:16pm

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jun 24, 2024
@dosubot dosubot bot added the Backend label Jun 24, 2024
@aakrem aakrem added the db label Jun 24, 2024
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Jun 25, 2024
@aybruhm aybruhm changed the title Cloud migration/beanie to sqlalchemy [cloud related] migration/beanie to sqlalchemy Jun 25, 2024
@aybruhm aybruhm requested review from mmabrouk and aakrem June 25, 2024 12:37
@aybruhm
Copy link
Member Author

aybruhm commented Jul 2, 2024

Hi @aakrem, did you create a PR from this PR and merge it into postgres-migration branch PR? I can find this PR commits in postgres-migration branch.

The following commits in the screenshot are from this PR (I am commenting from):

image

The following commits in the screenshot are from postgres-migration branch PR:

image

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 3, 2024
@aakrem aakrem merged commit 65fe52b into postgres Jul 3, 2024
8 of 10 checks passed
@aakrem aakrem deleted the cloud-migration/beanie-to-sqlalchemy branch July 3, 2024 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backend db lgtm This PR has been approved by a maintainer size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants