-
-
Notifications
You must be signed in to change notification settings - Fork 40
This issue refers to issue #9, to create student medical history schema. #24
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
This issue refers to issue #9, to create student medical history schema. #24
Conversation
Hitansh159
left a comment
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.
just small changes
models/std_med_history.js
Outdated
| @@ -0,0 +1,14 @@ | |||
| import { Schema } from 'mongoose'; | |||
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.
no need for this import
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.
Done
models/std_med_history.js
Outdated
| import { Schema } from 'mongoose'; | ||
| import { connector } from './databaseUtil'; | ||
|
|
||
| const studentMedicalSchema= new Schema({ |
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.
studentMedicalSchema will be just a object no need for new Schema
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.
Done
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.
restore it to its previous state
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.
Done
TejasNair9977
left a comment
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.
other than these changes, it looks good to me
models/std_med_history.js
Outdated
| bloodGroup:{type:String, required:true}, | ||
| pastMedicalHistory:{type:String, required:true}, | ||
| immunisationHistory:{type:String, required:true}, | ||
| chronicMedicalConditions:{type:String, required:true}, |
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.
this is not required
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.
Done
models/std_med_history.js
Outdated
| parentsContact:{type:String, required:true}, | ||
| relativeContacts:{type:String, required:true}, |
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.
can you make these contact fields as numbers
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.
Done
models/std_med_history.js
Outdated
| @@ -0,0 +1,13 @@ | |||
| import { connector } from './databaseUtil'; | |||
|
|
|||
| const studentMedicalSchema=({ | |||
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.
remove the extra brackets
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.
Done
TejasNair9977
left a comment
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.
looking good!
No description provided.