Rama-Ricardo #14

Merged
alex merged 8 commits from Rama-Ricardo into main 2024-09-04 17:53:01 +00:00
Member

Funcionalidades: Navegacion entre paginas con los botones previous-next, ordenar ascendente-descendente

Funcionalidades: Navegacion entre paginas con los botones previous-next, ordenar ascendente-descendente
jesus.pelaez added 2 commits 2024-08-28 19:09:26 +00:00
jesus.pelaez requested review from alex 2024-08-28 19:09:44 +00:00
jesus.pelaez added this to the pfs-intra project 2024-08-28 19:09:56 +00:00
jesus.pelaez added 1 commit 2024-08-28 19:37:19 +00:00
alex requested changes 2024-08-29 19:44:32 +00:00
Dismissed
@ -51,0 +67,4 @@
return button;
}
private void configureGrid() {
grid = new VGrid<>(Employee.class);
Owner

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

asi ya no necesitamos crear los botones y navegacion manualmente.

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 asi ya no necesitamos crear los botones y navegacion manualmente.
jesus.pelaez added 1 commit 2024-08-30 18:54:55 +00:00
jesus.pelaez added 1 commit 2024-09-03 15:42:29 +00:00
alex requested changes 2024-09-03 18:44:15 +00:00
Dismissed
@ -25,3 +1,1 @@
public enum Status {
ACTIVE,
INACTIVE
package com.primefactorsolutions.model;
Owner

quitar espacios blancos al inicio de cada linea

quitar espacios blancos al inicio de cada linea
@ -20,2 +23,4 @@
}
private void initializePagedBase() {
if (pagedBase == null) {
Owner

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.

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.
@ -22,0 +30,4 @@
public List<Employee> findEmployees(int start, int pageSize, String sortProperty, boolean asc) {
initializePagedBase(); // Asegurarse de que pagedBase esté inicializada
System.err.println("findAll " + start + " " + pageSize + " sort " + sortProperty + " asc:" + asc );
Owner

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

usar logger en vez de system println. ver lombok @Slf4j https://projectlombok.org/features/log
jesus.pelaez added 1 commit 2024-09-04 00:00:33 +00:00
jesus.pelaez added 1 commit 2024-09-04 00:11:58 +00:00
jesus.pelaez added 1 commit 2024-09-04 16:44:57 +00:00
alex approved these changes 2024-09-04 17:52:32 +00:00
alex merged commit 3320a4e99d into main 2024-09-04 17:53:01 +00:00
alex referenced this issue from a commit 2024-09-04 17:53:02 +00:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: PFS/pfs-intra#14
No description provided.