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.

Sign up for GitHub

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

Jump to bottom

Complete schema #3

Open
rmil opened this issue Jul 15, 2020 · 5 comments
Open

Complete schema #3

rmil opened this issue Jul 15, 2020 · 5 comments
Assignees
@rmil
Labels
enhancement New feature or request
Milestone
Release Candidate

Comments

@rmil
Copy link
Member

rmil commented Jul 15, 2020

Need to finish the proposed schema, make the migrations as well
@rmil rmil added the enhancement New feature or request label Jul 15, 2020
@rmil rmil added this to the Release Candidate milestone Jul 15, 2020
@rmil rmil self-assigned this Jul 15, 2020
@rmil
Copy link
Member Author

rmil commented Jul 18, 2020

So the schema for videos is okay-ish, I'll need to add the foreign key support for users and decide on how the tree of series is going to look but I'm happy-ish with it. Also, I'm thinking I'll make more columns NOT NULL since it's easier to deal with outside the database. But migrations are cool as a cucumber except for series nesting but thats to be decided. All the work should be available in b88d121.

@rmil
Copy link
Member Author

rmil commented Jul 18, 2020

Just noticed that for the video files, we have the vidserv copy which I don't believe was public? So I'll need to update the migration to support that / check for anything else similar.

@rmil
Copy link
Member Author

rmil commented Jul 27, 2020

I think I'll add COALESCE to some of the name fields so I can aggregate them?

@rmil
Copy link
Member Author

rmil commented Nov 18, 2020

Looking online, I might remove ON UPDATE CASCADE. Looking at this for example it seems alright to use the on delete but on update not so much. Perhaps for non-pk foreign fields?

@rmil
Copy link
Member Author

rmil commented Jan 17, 2021

Migrated officerships

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rmil rmil

Labels
enhancement New feature or request
Projects
None yet
Milestone
Release Candidate
Development

No branches or pull requests
1 participant
@rmil