-
Notifications
You must be signed in to change notification settings - Fork 0
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
add a description of the basic entities of the database #7
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I love the diagram!
Two things to think about:
- Note that supabase automatically creates a user table. It contains User UID, Display Name, etc. but has no location attribute. I think we should store the locations of our users in a separate table, which would contain the User UID and the location. Maybe we can also drop the location for now and add it later once we implement this feature...
- How do we know if an event is organized by an association or not ? I could be that a user is a committee member of an association but want's to create an event which is not tied to this association. Or the user is a member of multiple associations, to which one the event is linked to ? I think we can simply add to the event a FK to the association and then ensure that the organizer is actually a committee member of this association.
I agree, let's not include hypothetical things for now, but rather extend later on once we really need a field for a given feature. |
Thanks for the review,
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
daf037d
to
bc71edd
Compare
|
Basic description of the database for the app.
Created a mermaid diagram and a high level description of the entities.
fixes #5