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

refactor: migrate from Sqlite to MongoDB #85

Merged
merged 33 commits into from
Nov 3, 2023
Merged

refactor: migrate from Sqlite to MongoDB #85

merged 33 commits into from
Nov 3, 2023

Conversation

natelandau
Copy link
Owner

@natelandau natelandau commented Nov 3, 2023

Major refactor to migrate from SQLite to MongoDB. This has significantly reduced complexity in the code-base and (hopefully) eased future development

@natelandau natelandau merged commit 7994d1d into main Nov 3, 2023
4 checks passed
@natelandau natelandau deleted the mongodb branch November 3, 2023 21:28
natelandau added a commit that referenced this pull request Nov 4, 2023
* feat: migrate data from sqlite to mongo
* feat: migrate cogs
* fix: move add_roll_result_thumb to misc
* refactor: reorganize modules
* test: improve coverage
* fix: pass linters
* ci: add mongo to github ci
* build(deps): bump dependencies
* test: mock call to fakeuser.me
* test: use polyfactory factories for db models
* ci: fix poetry install
* ci: remove python 3.12 and refactor CI workflows
* refactor: move init database to utility function
* fix: prepare for deployment
natelandau added a commit that referenced this pull request Nov 4, 2023
* feat: migrate data from sqlite to mongo
* feat: migrate cogs
* fix: move add_roll_result_thumb to misc
* refactor: reorganize modules
* test: improve coverage
* fix: pass linters
* ci: add mongo to github ci
* build(deps): bump dependencies
* test: mock call to fakeuser.me
* test: use polyfactory factories for db models
* ci: fix poetry install
* ci: remove python 3.12 and refactor CI workflows
* refactor: move init database to utility function
* fix: prepare for deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant