refs #6802 Clientes-gestionados-por-equipos #1320

Merged
jgallego merged 34 commits from 6802-Clientes-gestionados-por-equipos into dev 2025-03-13 17:09:30 +00:00
Owner
No description provided.
jgallego added 2 commits 2025-01-31 14:49:04 +00:00
jgallego changed title from 6802-Clientes-gestionados-por-equipos to refs #6802 Clientes-gestionados-por-equipos 2025-01-31 14:49:15 +00:00
jgallego added 3 commits 2025-02-07 15:06:44 +00:00
jgallego added 13 commits 2025-02-26 07:47:40 +00:00
jgallego requested review from alexm 2025-03-06 06:38:58 +00:00
jgallego added 10 commits 2025-03-06 06:39:26 +00:00
alexm reviewed 2025-03-06 07:10:51 +00:00
@ -99,7 +99,6 @@ globals:
file: File
selectFile: Select a file
copyClipboard: Copy on clipboard
salesPerson: SalesPerson
Member

Se gastara en altres llocs no?

Se gastara en altres llocs no?
Author
Owner

El concept SalesPerson practicamente desapareix, es queda a nivell de rol

El concept SalesPerson practicamente desapareix, es queda a nivell de rol
jgallego marked this conversation as resolved
alexm reviewed 2025-03-06 07:12:46 +00:00
@ -72,2 +71,2 @@
:worker-id="entity.client?.salesPersonFk"
/>
<span class="link">
{{ entity?.client?.department?.name || '-' }}
Member

Gastar dashIfEmpty pr al '-'
Pero vegent que la estructura de text + DepartmentDescriptorProxy se repetix igual crearia component paregut a VnUserLink (O ficat altre redmine de refactor)

Gastar dashIfEmpty pr al '-' Pero vegent que la estructura de text + DepartmentDescriptorProxy se repetix igual crearia component paregut a VnUserLink (O ficat altre redmine de refactor)
Author
Owner
https://redmine.verdnatura.es/issues/8764
alexm reviewed 2025-03-06 07:14:13 +00:00
@ -71,4 +67,0 @@
'isFreezed',
'isTaxDataChecked',
],
include: {
Member

Falta llevar on se gasta .salesPersonUser

Falta llevar on se gasta `.salesPersonUser`
Author
Owner

salesPersonUser ja no apareix enel projecte

salesPersonUser ja no apareix enel projecte
alexm requested changes 2025-03-06 07:15:40 +00:00
Dismissed
alexm left a comment
Member

Lo demes ho vec be, si vols aprove i se pot mirar despres

Lo demes ho vec be, si vols aprove i se pot mirar despres
jgallego added 2 commits 2025-03-13 06:48:19 +00:00
jgallego requested review from alexm 2025-03-13 06:48:44 +00:00
alexm approved these changes 2025-03-13 06:59:50 +00:00
Dismissed
alexm added 1 commit 2025-03-13 07:00:06 +00:00
alexm dismissed alexm’s review 2025-03-13 07:00:07 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

alexm approved these changes 2025-03-13 07:00:29 +00:00
Dismissed
Member

Crec que com la rama ja te més de un mes, jenkins no tira els tests.
Hi ha que tancar esta PR, i obrir altra (es pot reutilitzar la rama)

Crec que com la rama ja te més de un mes, jenkins no tira els tests. Hi ha que tancar esta PR, i obrir altra (es pot reutilitzar la rama)
jgallego added 2 commits 2025-03-13 09:06:01 +00:00
jgallego dismissed alexm’s review 2025-03-13 09:06:01 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

jgallego added 1 commit 2025-03-13 09:09:55 +00:00
jgallego merged commit 7129c236dd into dev 2025-03-13 17:09:30 +00:00
jgallego deleted branch 6802-Clientes-gestionados-por-equipos 2025-03-13 17:09:30 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
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-front#1320
No description provided.