alex alex
  • Joined on 2024-05-04
alex suggested changes for PFS/pfs-intra#27 2024-09-17 17:46:15 +00:00
En-desarrollo
alex pushed to main at PFS/pfs-intra 2024-09-08 14:10:11 +00:00
1f429221d4 ldap employee updates
alex pushed to main at PFS/pfs-intra 2024-09-08 13:08:43 +00:00
ccb3629409 #4-Registro-de-Información-Personal (#16)
alex merged pull request PFS/pfs-intra#16 2024-09-08 13:08:42 +00:00
#4-Registro-de-Información-Personal
alex commented on pull request PFS/pfs-intra#16 2024-09-06 14:55:20 +00:00
#4-Registro-de-Información-Personal

no es necesario agregar esto porque ya existe el boton edit que navega a la vista edit.

alex commented on pull request PFS/pfs-intra#16 2024-09-06 14:55:20 +00:00
#4-Registro-de-Información-Personal

no es una buena practica acceder a las dependencies del service. Veo que aca el problema es que la clase EmployeeService tiene anotacion @Data. Porfavor quitar la anotacion @Data de EmployeeService y crear metodo findAllEmployees si es necesario.

alex suggested changes for PFS/pfs-intra#16 2024-09-06 14:55:20 +00:00
#4-Registro-de-Información-Personal
alex pushed to main at PFS/pfs-intra 2024-09-06 00:59:22 +00:00
c91e4ea1a5 #4-Registrar-empleado (#15)
alex merged pull request PFS/pfs-intra#15 2024-09-06 00:59:20 +00:00
#4-Registrar-empleado
alex approved PFS/pfs-intra#15 2024-09-06 00:59:04 +00:00
#4-Registrar-empleado
alex commented on pull request PFS/pfs-intra#15 2024-09-05 22:47:09 +00:00
#4-Registrar-empleado

el formato de la ruta deberia ser al reves:

alex suggested changes for PFS/pfs-intra#15 2024-09-05 22:47:09 +00:00
#4-Registrar-empleado
alex pushed to main at PFS/pfs-intra 2024-09-04 17:53:02 +00:00
3320a4e99d Rama-Ricardo (#14)
alex merged pull request PFS/pfs-intra#14 2024-09-04 17:53:01 +00:00
Rama-Ricardo
alex approved PFS/pfs-intra#14 2024-09-04 17:52:32 +00:00
Rama-Ricardo
alex commented on pull request PFS/pfs-intra#14 2024-09-03 18:44:15 +00:00
Rama-Ricardo

quitar espacios blancos al inicio de cada linea

alex commented on pull request PFS/pfs-intra#14 2024-09-03 18:44:15 +00:00
Rama-Ricardo

usar logger en vez de system println. ver lombok @Slf4j https://projectlombok.org/features/log

alex commented on pull request PFS/pfs-intra#14 2024-09-03 18:44:15 +00:00
Rama-Ricardo

no se puede guardar en un campo de la clase porque la clase es un singleton (si hay varios llamados van a haber problemas de concurrencia). solucion: no usar un campo en la clase y simplemente retornar toda la lista cada vez sin guardar en un campo.

alex suggested changes for PFS/pfs-intra#14 2024-09-03 18:44:15 +00:00
Rama-Ricardo
alex commented on pull request PFS/pfs-intra#14 2024-08-29 19:44:32 +00:00
Rama-Ricardo

podemos usar el componente PagingGrid de la llibreria Viritin (ya esta incluida). Ejemplo: https://addons.dokku1.parttio.org/paginggrid y codigo https://github.com/parttio/addon-demos/blob/main/src/main/java/org/example/views/PagingGridView.java