verdnatura/salix
verdnatura
/
salix
3
4
Fork 0
Code Issues Pull Requests 64 Releases Wiki Activity

feat: refs #8167 update canBeInvoiced method to include active status check and improve test cases #3323

Merged
jgallego merged 4 commits from 8167-incoiceOutCheckIsActive into dev 2024-12-30 13:37:31 +00:00
Conversation 2 Commits 4 Files Changed 6 +48 -57
jgallego commented 2024-12-19 06:08:45 +00:00
Owner
Copy Link
No description provided.
jgallego added 1 commit 2024-12-19 06:08:46 +00:00
jgallego requested review from jorgep 2024-12-19 06:09:17 +00:00
jorgep commented 2024-12-19 09:12:05 +00:00
Member
Copy Link

image

La descripción de la tarea pone: Si un cliente está desactivado muestra este error, debería mostrar un error tipo el cliente está desactivado. ¿Con los cambios que has hecho seguro que sale el mensaje de error que se pide? He hecho una prueba y no veo nada, el front está llamando a invoiceOut/clientsToInvoice

Y por lo que veo el unico error que puede mostrar es el del address

image

![image](/attachments/4832f8ba-79b6-4569-8681-681bd88d8ebc) La descripción de la tarea pone: **Si un cliente está desactivado muestra este error, debería mostrar un error tipo el cliente está desactivado.** ¿Con los cambios que has hecho seguro que sale el mensaje de error que se pide? He hecho una prueba y no veo nada, el front está llamando a **invoiceOut/clientsToInvoice** Y por lo que veo el unico error que puede mostrar es el del address ![image](/attachments/61594e58-89f9-4375-b438-5e402404fece)
image.png
59 KiB
image.png
88 KiB
jorgep requested changes 2024-12-19 09:12:39 +00:00
Dismissed
jorgep left a comment
Member
Copy Link

.

.
jgallego added 2 commits 2024-12-29 08:39:04 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
98e2483565 feat: refs #8167 update locale and improve invoicing logic with error handling
jgallego commented 2024-12-29 08:40:23 +00:00
Author
Owner
Copy Link

Wow, bien visto. He aprovechado para no replicar las comprobaciones y usar el back que ya las hacia

Wow, bien visto. He aprovechado para no replicar las comprobaciones y usar el back que ya las hacia
jgallego requested review from jorgep 2024-12-29 08:40:27 +00:00
jorgep approved these changes 2024-12-30 08:49:54 +00:00
jorgep left a comment
Member
Copy Link

Igual habría que refactorizar el front, en otra tarea.

Igual habría que refactorizar el front, en otra tarea.
jgallego added 1 commit 2024-12-30 13:35:20 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
a26de63660 Merge branch 'dev' into 8167-incoiceOutCheckIsActive
jgallego merged commit be50c91299 into dev 2024-12-30 13:37:31 +00:00
jgallego deleted branch 8167-incoiceOutCheckIsActive 2024-12-30 13:37:31 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
jorgep
Labels
Clear labels   
CR / Tests passed

Unit tests and e2e passed

  
e2e

Implement e2e

  
Missing tests

Missing tests

  
Needs refactor

   
Simple

Simple check / No CR needed

No Label
CR / Tests passed
e2e
Missing tests
Needs refactor
Simple
Milestone
Clear milestone
No items
No Milestone
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: verdnatura/salix#3323
No description provided.
Delete Branch "8167-incoiceOutCheckIsActive"

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?