-
Notifications
You must be signed in to change notification settings - Fork 30
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
Models for project and user #25
Comments
const RepoSchema=new express.Schema({ |
As for user }); |
@highskillzz As per the designs, I think we are including all those information, along with more options such as browser support. |
The main point is are we authenticating the normal user who is going to view the projects only? Apart from that, the project owner will be having access to the repos he or she built. What we can do is while designing the model schema, we should remember that the projects are related to the project owner. So a project owner can have the following type of schema:
This is one suggestion for schema. If we follow the traditional approach of schema design, we can keep a user schema and a projects schema as shown by @highskillzz . But we need to choose the fields carefully as we need to get them implemented for the 1st draft too. Then later on we can add more such fields and make it even better. Let me know what you guys think? |
The authentication for the user would be required only under the circumstances that the user will comment on the repo. Else there would be no point in authenticating the user because he will only view the project unless as mentioned he will comment too. Shall I open a pull request as a work in progress for this then? |
Guys for the model for the user and the project submitted, how about
For each project the owner can give in their github project link, its working link , its description. contributors and then we can show it in the frontend. hence the model will be like:-
name, github link, hosted link, description, owner, comments(array), moderators
As for the user
name,password,username,projects(array)
What are your thoughts?
The text was updated successfully, but these errors were encountered: