-
Notifications
You must be signed in to change notification settings - Fork 12
REC-136: Efectores y cuil paciente #2150
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
base: master
Are you sure you want to change the base?
Conversation
9ea8555 to
0000035
Compare
33f7a1b to
bd1b4cb
Compare
| profesion: String, | ||
| matricula: Number, | ||
| especialidad: String, | ||
| efector: { |
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.
Me hace ruido que el dato del efector quede dentro del sub-esquema de profesional. De todos modos, me surge la duda si no se podría haber utilizado los datos de organización que ya contiene la receta (hoy en día las recetas recibidas desde recetar en organizacion sólo guardan { nombre: "recetar"} y ese dato también se encuentra en otros campos como origenExterno.
Incluso chusmeando un poco más el esquema de api veo que está la variable dentro de estados de la receta "organizacionExterna" no se usa para nada. No se si la usaría dentro de los estados, pero al menos se podría aprovecha para borrar (igual esto último tampoco es taaan necesario, solo se me ocurría como boys scouting)
| especialidad: especialidades, | ||
| matricula: matriculaGrado | ||
| matricula: matriculaGrado, | ||
| efector: profRecetar.efector || null |
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.
idem anterior
bd1b4cb to
e91be39
Compare
Requerimiento
REC-136
Funcionalidad desarrollada
UserStories llegó a completarse
Requiere actualizaciones en la base de datos
Tarea vinculada a pr Recetar-api y Recetar-app
andes/recetar-api#67
andes/recetar-app#105
andes/app#3283