Skip to content
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

(FEAT) ABM de restricción por paciente #68

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

JuanIRamirez
Copy link

@JuanIRamirez JuanIRamirez commented Feb 9, 2024

Assignee: Juan Ramirez2
ready

Requerimiento

https://proyectos.andes.gob.ar/browse/MONIT-60

Funcionalidad desarrollada

  1. Nuevo Módulo restriccion-huds
  2. Se muestra listado de Usuarios.
  3. Al seleccionar un usuario se cargan los pacientes restringidos.
  4. Por cada paciente se puede adjuntar archivos.
  5. Los archivos se guardan en las colecciones: RestriccionHUDSStore.files y RestriccionHUDSStore.chunks.

UserStory llegó a completarse

  • Si
  • No
  • No corresponde

Requiere actualizaciones en la base de datos

  • Si
  • No

Requiere actualizaciones en la API

Requiere actualizaciones en andes-test-integracion

  • Si
  • No

Copy link
Contributor

@negro89 negro89 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OBSERVACIONES:

  1. Se duplicó el componente paciente-buscar (no debería). Queda pendiente corregir esto y ver que queden funcionando bien todas las referencias a ambos componentes.
  2. La foto de los usuarios en el listado no debería aparecer.

@@ -0,0 +1,7 @@
export interface IContacto {
tipo: any;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Como tipo hace referencia a "celular, fijo o email" se puede cambiar su declaración por string en vez de any

@@ -0,0 +1,84 @@
import { IUbicacion } from './IUbicacion';
// import { IMatricula } from './IMatricula';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remover comentario!

inicio: Date;
baja: {
motivo: String;
fecha: any;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cambiar any por Date

@@ -0,0 +1,8 @@
// import { IPacienteMatch } from '../interfaces/IPacienteMatch.inteface';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eliminar comentario


export interface PacienteBuscarResultado {
err: any;
pacientes: any[];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Como es un array de pacientes se puede remplazar por Ipaciente[]

Comment on lines +7 to +14
interface PacienteEscaneado {
documento: string;
apellido: string;
nombre: string;
sexo: string;
fechaNacimiento: Date;
scan: string;
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Si esta interfaz no se utiliza entonces eliminarla.

Comment on lines +8 to +9
// import { NombrePipe } from '../pipes/nombre.pipe';
// import { EdadPipe } from '../pipes/edad.pipe';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eliminar estas lineas de código!

/**
* RegEx para validar nombres y apellidos.
*/
public nombreRegEx = /^[a-zA-ZàáèéìíòóùúüñÀÁÈÉÌÍÒÓÙÚÑ']+( [a-zA-ZZàáèéìíòóùúüñÀÁÈÉÌÍÒÓÙÚÑ']+)*$/;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Parece que esta expresión regular no se estaría usando en ningún lugar.

Comment on lines +17 to +21
// organizaciones(): Observable<any> {
// return this.server.get('/modules/gestor-usuarios/organizaciones', {}).pipe(
// cacheStorage({ key: 'organizaciones-permisos' })
// );
// }
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eliminar lineas de código comentadas

Comment on lines +97 to +118
* Metodo post. Inserta un nuevo profesional
* @param {IProfesional} profesional
*/
// post(profesional: IProfesional): Observable<IProfesional> {
// return this.server.post(this.profesionalUrl, profesional); // ...using post request
// }

// saveFirma(firma) {
// return this.post(firma);
// }

// saveProfesional(profesionalModel: any) {
// return profesionalModel.id ? this.server.patch(`${this.profesionalUrl}/${profesionalModel.id}`, profesionalModel) : this.server.post(this.profesionalUrl, { profesional: profesionalModel });
// }

// validarProfesional(body): Observable<any> {
// return this.server.post(this.profesionalUrl + '/validar', body);
// }

// actualizarProfesional(body, options?: Options): Observable<any> {
// return this.server.put(this.profesionalUrl + '/actualizar', body, options);
// }
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eliminar estas lineas de codigo comentadas al igual que la linea 5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants