PFS/pfs-intra
4
0
Fork 0
Code Issues 32 Pull Requests Actions Packages Projects 2 Releases Wiki Activity

WIP: Rama-Melina #4 y #5 #13

Closed
melina.gutierrez wants to merge 3 commits from Rama-Melina into main
pull from: Rama-Melina
merge into: PFS:main
Conversation 0 Commits 3 Files Changed 3 +145 -11
melina.gutierrez commented 2024-08-28 11:05:14 +00:00
Member
Copy Link

Se realizaron cambios en EployeeView.java : crear la información general del empleado y también el botón de guardar un empleado.

Se realizaron cambios en EployeeView.java : crear la información general del empleado y también el botón de guardar un empleado.
melina.gutierrez added 3 commits 2024-08-28 11:05:15 +00:00
alex requested changes 2024-08-28 12:47:48 +00:00
alex left a comment
Owner
Copy Link

revisar el modelo y ver que se guarden los campos en la db

revisar el modelo y ver que se guarden los campos en la db
src/main/java/com/primefactorsolutions/views/EmployeeView.java
@ -37,3 +136,4 @@
ss, emergencyCName, emergencyCAddress, emergencyCPhone, emergencyCEmail);
setSavedHandler((SavedHandler<Employee>) employee -> {
final Employee saved = employeeService.createOrUpdate(employee);
alex commented 2024-08-28 12:47:22 +00:00
Owner
Copy Link

parece que los campos de emergency contact no estan definidos en la clase model Employee. Se deberian crear esos campos para que puedan ser guardados en al DB.

podrias ir viendo de agregar campo por campo y ver si se guardan en la db cuando se hace el save.

parece que los campos de emergency contact no estan definidos en la clase model Employee. Se deberian crear esos campos para que puedan ser guardados en al DB. podrias ir viendo de agregar campo por campo y ver si se guardan en la db cuando se hace el save.
melina.gutierrez closed this pull request 2024-09-02 17:47:39 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
alex
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
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#13
No description provided.
Delete Branch "Rama-Melina"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?