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

Epic: Define (Ecto) PostgreSQL Database Schema #15

Open
2 of 3 tasks
nelsonic opened this issue Feb 19, 2018 · 5 comments
Open
2 of 3 tasks

Epic: Define (Ecto) PostgreSQL Database Schema #15

nelsonic opened this issue Feb 19, 2018 · 5 comments

Comments

@nelsonic
Copy link
Member

nelsonic commented Feb 19, 2018

All the "Meta Data" (anything that is not an issue comment, which is Markdown/"Rich Text")
of issues will be stored in a PostgreSQL Database.

With reference to the GitHub API for users, issues and comments,
we need to create the Ecto (Database) Schema for the three DB tables.
We are going to "normalise" the data and let Ecto construct the queries.

Our aim is to match the field names in the GitHub API exactly
so we keep an exact copy the data that people can easily download
if they want to have a Local Copy. (don't worry, we aren't building the download feature yet!)

@Cleop
Copy link
Member

Cleop commented Feb 27, 2018

Relates to #5

In the long run we might want these two tables to help us display our UI better but for now if people are accessing their comments directly from the issue then they wouldn't be necessary.

Organisations
ID
Organisation name
Repositories
ID
Repo name

Do people think we should include them in our schema now or add them later if necessary?

@Cleop Cleop removed the in-progress label Feb 27, 2018
@nelsonic
Copy link
Member Author

@Cleop if we know the desired schema up-front then we should create it as close to the "long term" as possible.

@Cleop
Copy link
Member

Cleop commented Feb 28, 2018

Do we need the avatar url in our schema for displaying the user's photo next to their comments?
image

@nelsonic
Copy link
Member Author

@Cleop yeah, we should save the "avatar_url".
For now please assume we are storing all the info for a user.
Thanks.

@Cleop Cleop mentioned this issue Mar 2, 2018
3 tasks
@Cleop
Copy link
Member

Cleop commented Mar 7, 2018

Still to do #46

@Cleop Cleop removed their assignment Mar 7, 2018
@Cleop Cleop removed the discuss label Mar 7, 2018
@nelsonic nelsonic added this to the MVP milestone Mar 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants