Replies: 20 comments 6 replies
-
Hi, Can you elaborate on what is the parent comment FK is? |
Beta Was this translation helpful? Give feedback.
-
Hi, This model allows for "reply on top of reply" or a comment to a comment, and was shown in the previous Beyond session by Sim. |
Beta Was this translation helpful? Give feedback.
-
Hi, I just found out that "PREFERENCES"'s "location" should also be an enum:
So I made an edit to the graph here: |
Beta Was this translation helpful? Give feedback.
-
I have deleted the relationship between personal profile and preferences as only the job-offer should have its different preferences and a job-seeker will filter out different job-offers according to it's preferences. |
Beta Was this translation helpful? Give feedback.
-
I think that the user should only contain the relevant attributes for logging in (such as credentialsId), |
Beta Was this translation helpful? Give feedback.
-
Also, since we have the user model in Django (https://docs.djangoproject.com/en/4.0/ref/contrib/auth/), |
Beta Was this translation helpful? Give feedback.
-
@amittcohen I think you're right and the birth date should move to the personal profile.
I'm not sure why we even decided to separate it in the first place. |
Beta Was this translation helpful? Give feedback.
-
Good idea.
I added the credentials table as an offer for us to discuss. I think it is helpful if we choose to use credentials from Google / Facebook signups. |
Beta Was this translation helpful? Give feedback.
-
Updated post and comment, also added likes: |
Beta Was this translation helpful? Give feedback.
-
@paOmer @amittcohen @taljacob2 Also, wanted to make sure that "preferencesID" is not part of the personal profile model since it's related to posts and jobs only. |
Beta Was this translation helpful? Give feedback.
-
Deleted birthday from the user model and added it to personal profile- I've seen that we wrote string on the data type but it's preferred to have it as date, right? |
Beta Was this translation helpful? Give feedback.
-
I removed the CREDENTIALS table now. |
Beta Was this translation helpful? Give feedback.
-
|
Beta Was this translation helpful? Give feedback.
-
Edited "MESSAGE"'s properties to be similar to "COMMENT" and "POST". |
Beta Was this translation helpful? Give feedback.
-
Found it! It should be Updated the ERD: |
Beta Was this translation helpful? Give feedback.
-
@taljacob2 Thank you! I will also change it on the models.py file. |
Beta Was this translation helpful? Give feedback.
-
Some thoughts about the job_board's data models:
|
Beta Was this translation helpful? Give feedback.
-
Updated the Location table: |
Beta Was this translation helpful? Give feedback.
-
UPDATE
|
Beta Was this translation helpful? Give feedback.
-
This is the ERD @DeanBiton posted on our slack:
I converted this image to mermaid markdown, so now we can all edit the graph together.
You are welcome to make changes to this graph. (You can view the simple guide about how to edit mermaid ERD graphs here)
⬆️ ⬆️ ⬆️ Press on the graph above to make changes on it. ⬆️ ⬆️ ⬆️
By the way, once you have pressed on the graph, you can view it on full screen, by pressing the "View" button in the top right corner:
Edits Made Since @DeanBiton's Post On Our Slack, That Require Discussions
I added a "CREDENTIALS" table to be referenced from "USER" (This is a good feature if we want to use "Google / Facebook Signup" in the future). Please comment here if this is something we should go for or not.
I split the "JOB-TYPE" and "YEARS-OF-EXPERIENCE" to their own tables, because they both represent enums with limited option sets.
We divided the features between us, and I am assigned to work on the "Chat". I need assistance from @Yarboa @kasemAlem @rrasouli about the relationships for the "CHAT" (Can you please comment here your thoughts about the "CHAT" table?). I tend to think this is a "many to many" relationship, so I edited the ERD version @DeanBiton has posted on our slack to what I think we should do, but I am not sure about this. In the new edited version of the graph, it is possible that a chat may bundle multiple users together for a conversation (to support unlimited number of users).
Beta Was this translation helpful? Give feedback.
All reactions