-
Notifications
You must be signed in to change notification settings - Fork 189
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 Planet Feed to Supabase db #2860
Comments
The code in https://github.com/Seneca-CDOT/telescope/tree/master/tools/migrate can help with this. In our discussions today, we said that we'd untangle the idea of a user from feeds, so that we can create feeds in our database, but not necessarily associate them with a user. This could be done via an optional foreign key value that links to a user; or we could use a third table to link |
For the sake of simplicity, I suggest we go with the option that @DukeManh proposed: an optional field. For this to work, we need to keep two facts:
The I suggest we violate some rules of database design for the sake of simplicity and add a field that stores the author name (call it |
I like this. We would end up with:
We still need to add some more data here. Our concept of a For example, we still need:
Then we have some other things to consider:
I'm not sure what else I'm missing, but we'll have to deal with all of this in various PRs. |
For this, we may name it as the "source" of the feed. We can create a table called
The
I agree. There can be a simple |
I also want to inform that Postgres tables can contain Array type and JSON type and we probably don't have to create many tables that reference each other. |
What would you like to be added:
Part of the Supabase implementation, we need to move the list of feeds from the Seneca CDOT wiki page to new table(s)
We need to figure out the schema to store and relate the tables.
Note: Duc has made a PR regarding this "Handle Telescope user registration and login", but we plan to split the PR into smaller manageable parts.
Why would you like this to be added:
To further progress implementing Supabase to Telescope
The text was updated successfully, but these errors were encountered: