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

Migrate the codebase to use Supabase instead of MySQL #6

Closed
wants to merge 8 commits into from

Conversation

sweep-ai[bot]
Copy link

@sweep-ai sweep-ai bot commented Sep 12, 2023

Description

This PR migrates the codebase from MySQL to Supabase as the database provider. It includes changes to the database configuration, ORM models, service layer, and test data helper. The goal is to ensure that the application continues to function as expected after the migration.

Summary of Changes

  • Updated the database configuration in app/config.py to connect to Supabase instead of MySQL.
  • Modified the ORM models in app/entities/orm/orm_model.py to work with Supabase.
  • Updated the service layer in app/api/endpoints/users/users_service.py to use Supabase's API for database operations.
  • Rewrote the test data helper in tests/data_helper.py to work with Supabase.
  • Removed the Docker configuration for the database in db/docker-compose.yml.
  • Removed the MySQL initialization script in db/init/10.my_db.sql.
  • Removed the MySQL configuration file in db/my.cnf.

Please review the changes and ensure that the application functions correctly with Supabase as the database provider.

Fixes #5.


To checkout this PR branch, run the following command in your terminal:

git checkout sweep/migrate-to-supabase

🎉 Latest improvements to Sweep:


💡 To get Sweep to edit this pull request, you can:

  • Leave a comment below to get Sweep to edit the entire PR
  • Leave a comment in the code will only modify the file
  • Edit the original issue to get Sweep to recreate the PR from scratch

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.

Sweep: Migrate the codebase to use Supabase instead of MySQL
1 participant