verdnatura/salix
verdnatura
/
salix
3
5
Fork 0
Code Issues Pull Requests 62 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 +3 -3
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 36297009e1 - Show all commits

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

@ -87,10 +87,10 @@ module.exports = Self => {
[itemFk, 2]);
// if (filter.where.alertLevel) {
jgallego marked this conversation as resolved Outdated
jgallego commented 2025-01-31 14:31:41 +00:00
Outdated
Review
Copy Link

aqui falta el alias de la tabla, probablemente t.warehouse

aqui falta el alias de la tabla, probablemente t.warehouse
👍 1
jgallego commented 2025-02-06 12:09:12 +00:00
Outdated
Review
Copy Link

, COUNT(*) hasObservation

Hemos invertido este booleano, ahora hay que mostrar el icono si aqui hay valor

, COUNT(*) hasObservation Hemos invertido este booleano, ahora hay que mostrar el icono si aqui hay valor
// stmt.merge({
// sql: `AND ts.alertLevel=?`, params: ['FREE']});
stmt.merge({
sql: `AND ${filter.where.alertLevel ? '' : 'NOT'} ts.alertLevel=?`, params: [0]});
// }
// stmt.merge(conn.makeWhere(filter));
// stmt.merge(conn.makeWhere(filter.where));
const sql = ParameterizedSQL.join([stmt], ';');
const result = await conn.executeStmt(sql, myOptions);
jgallego marked this conversation as resolved Outdated
jsegarra commented 2025-01-31 00:15:30 +00:00
Outdated
Review
Copy Link

Valores como estos lo dejamos así o lo movemos a variables/consultas?

Valores como estos lo dejamos así o lo movemos a variables/consultas?
jgallego commented 2025-02-05 06:42:52 +00:00
Outdated
Review
Copy Link

si es alertLevel hay que usar el codigo, no el numero, en este caso 'ON_PREPARATION' o 'DELIVERED'

si es alertLevel hay que usar el codigo, no el numero, en este caso 'ON_PREPARATION' o 'DELIVERED'
jsegarra commented 2025-02-05 06:49:13 +00:00
Outdated
Review
Copy Link

Okey, hago la modificación aqui y en el front para que se pueda enviar array desde el VnSelect

Okey, hago la modificación aqui y en el front para que se pueda enviar array desde el VnSelect
jsegarra commented 2025-02-05 23:33:18 +00:00
Outdated
Review
Copy Link

Resuelto

Resuelto
jsegarra commented 2025-02-06 00:07:28 +00:00
Outdated
Review
Copy Link

Y he creado un registro en TicketConfig con valor default a FREE (por si cambian de criterio a futuro)

Y he creado un registro en TicketConfig con valor default a FREE (por si cambian de criterio a futuro)
return result;