ER modeling #32
ER modeling #32
Conversation
Add new issue template
added the project board to the template
|
Nice job BK. Could be removed this file I believe your branch is not up to date. Please do a rebase to fix that! |
Doc/ER_diagram.md
Outdated
| STUDENT ||--|| CALENDAR : has | ||
| MENTOR ||--|| CALENDAR : has | ||
| MENTOR ||--|{ WEEKDAYS : tutors | ||
| ADMIN ||--o{ EVENTS : posts |
There was a problem hiding this comment.
add a string password field here
There was a problem hiding this comment.
The first section represents Entity relationships and each constitutes its own table. Do you mean to have a Password entity? or if you are referring to string password, let me know the table so I can add it@kirubeltadesse
There was a problem hiding this comment.
I understand. clicked at the wrong place by mistake :)
Doc/ER_diagram.md
Outdated
| @@ -0,0 +1,63 @@ | |||
| # This diagram represents the ER relationship diagram among the entities below | |||
There was a problem hiding this comment.
Add a title and make the rest of the information a paragraph: for better formatting.
| string Jnumber | ||
| string email | ||
| string role | ||
| string userName |
There was a problem hiding this comment.
Sorry, I mean add the password here. Next to the userName
Doc/ER_diagram.md
Outdated
| STUDENT ||--|| CALENDAR : has | ||
| MENTOR ||--|| CALENDAR : has | ||
| MENTOR ||--|{ WEEKDAYS : tutors | ||
| ADMIN ||--o{ EVENTS : posts |
There was a problem hiding this comment.
I understand. clicked at the wrong place by mistake :)
Doc/ER_diagram.md
Outdated
| int calendar_id PK | ||
| int weekday_id FK | ||
| } | ||
| WEEKDAYS { |
There was a problem hiding this comment.
Should we add an entry for 'week' here?
| STUDENT ||--|| CALENDAR : has | ||
| MENTOR ||--|| CALENDAR : has | ||
| USER { | ||
| int user_id PK |
There was a problem hiding this comment.
The datatype of user_id seems to be inconsistent. It is an int datatype here but on STUDENT, MENTOR and SIGNIN it seems to be of string data type
There was a problem hiding this comment.
@dinaolmelak this is closed PR?
There was a problem hiding this comment.
Gotcha, I have added these changes to my pull request #44
| USER { | ||
| int user_id PK | ||
| string fullName | ||
| string Jnumber |
There was a problem hiding this comment.
can we set this to camel case naming convention? (i.e string jNumber)
| string user_id FK | ||
| } | ||
| MENTOR { | ||
| int mentor_id PK |
There was a problem hiding this comment.
The data type of mentor_id is int here but under EVENTS it is a string.
| USER { | ||
| int user_id PK | ||
| string fullName | ||
| string Jnumber |
There was a problem hiding this comment.
Can we also set the data type of jNumber to int?
Description
This is basic ER relationship diagram that is based on UML diagram. It's dependent on MermaidJS
Fixes #19
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
It has been tested locally and on live editor. A link is included to help the developer replicate the same test
Test Configuration:
N/A
Checklist: