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

#6321 - Negative tickets #1945

Merged
jsegarra merged 146 commits from 6321_negative_tickets into dev 2025-02-11 08:45:33 +00:00
Conversation 0 Commits 146 Files Changed 27 +5 -4
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 6c0706cc56 - Show all commits

9
modules/ticket/back/methods/ticket/itemLackDetail.js
View File

@ -37,19 +37,20 @@ module.exports = Self => {
st.code, st.code,
t.id ticketFk, t.id ticketFk,
t.nickname, t.nickname,
t.shipped,
jgallego commented 2025-02-07 11:53:02 +00:00
Outdated
Review
Copy Link

este dos?

este dos?
jsegarra commented 2025-02-07 21:39:26 +00:00
Outdated
Review
Copy Link

Este 2 lo he puesto para que cuando filter.where no tenga scopeDays no de error la consulta, sin embargo, para ser lo mas agnóstico posible a quien lo llame, si no me lo pasas, asigno un 0 que es más lógico

Este 2 lo he puesto para que cuando filter.where no tenga scopeDays no de error la consulta, sin embargo, para ser lo mas agnóstico posible a quien lo llame, si no me lo pasas, asigno un 0 que es más lógico
jsegarra commented 2025-02-08 15:31:48 +00:00
Outdated
Review
Copy Link

Si lo ves correcto, te pido fusionar

Si lo ves correcto, te pido fusionar
jgallego commented 2025-02-10 14:25:22 +00:00
Outdated
Review
Copy Link

mejora aun, no pases nada si no pasan nada.

mejora aun, no pases nada si no pasan nada.
👍 1
t.hour, t.hour,
s.quantity, s.quantity,
ag.name, ag.name agName,
ts.alertLevel alertLevel, ts.alertLevel alertLevel,
st.name stateName, st.name stateName,
st.id stateId, st.id stateId,
s.itemFk itemFk, s.itemFk itemFk,
al.code alertLevelCode, al.code alertLevelCode,
z.name, z.name zoneName,
z.hour theoreticalhour, z.hour theoreticalhour,
cn.isRookie, cn.isRookie,
jgallego marked this conversation as resolved Outdated
jgallego commented 2025-02-05 06:36:06 +00:00
Outdated
Review
Copy Link

yo estoy cambiando los alertLevel, no podemos poner numero en el codigo, si no hay ..null

yo estoy cambiando los alertLevel, no podemos poner numero en el codigo, si no hay ..null
jsegarra commented 2025-02-05 14:42:30 +00:00
Outdated
Review
Copy Link

ok, lo reviso

ok, lo reviso
jsegarra commented 2025-02-06 11:49:50 +00:00
Outdated
Review
Copy Link

Lo he valorado con otros compañeros y esto de momento lo dejamos así

Lo he valorado con otros compañeros y esto de momento lo dejamos así
sc.saleClonedFk turno, IF(sc.saleClonedFk, 1, 0 ) as turno,
tr.saleFk peticionCompra, IF(tr.saleFk , 1, 0 ) as peticionCompra,
t.hour minTimed t.hour minTimed
FROM FROM
jsegarra marked this conversation as resolved Outdated
jgallego commented 2025-01-31 14:30:24 +00:00
Outdated
Review
Copy Link

es una chorrada pero comovas a cambiar cosas, no solemos poner los AS en los select

es una chorrada pero comovas a cambiar cosas, no solemos poner los AS en los select
vn.sale s vn.sale s
jsegarra marked this conversation as resolved Outdated
jgallego commented 2025-02-06 11:59:13 +00:00
Outdated
Review
Copy Link

tls.alertLevel

tls.alertLevel