#6321 - Negative tickets #1945

Open
jsegarra wants to merge 93 commits from 6321_negative_tickets into dev
Member
No description provided.
jsegarra added 1 commit 2024-01-20 11:30:12 +00:00
gitea/salix/pipeline/head This commit looks good Details
91f5ee3b93
refs #6321 feat: new remoteMethod
jsegarra added 1 commit 2024-01-22 19:29:53 +00:00
gitea/salix/pipeline/head This commit looks good Details
48d9a3934a
Merge branch 'dev' into 6321_negative_tickets
jsegarra added 3 commits 2024-01-23 10:15:49 +00:00
jsegarra added 2 commits 2024-01-29 08:56:10 +00:00
gitea/salix/pipeline/head There was a failure building this commit Details
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
1a21dda00b
refs #6321 feat itemLackDetail
jsegarra added 2 commits 2024-03-05 07:08:32 +00:00
jsegarra added 2 commits 2024-03-06 13:38:07 +00:00
jsegarra added 1 commit 2024-03-13 08:21:12 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
c8446eb9a1
refs #6321 perf: updatemethod
jsegarra added 2 commits 2024-03-14 14:27:14 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
6a12af2eb9
refs #6321 feat: add producerFk
jsegarra added 3 commits 2024-03-15 08:47:58 +00:00
jsegarra added 1 commit 2024-03-21 14:04:01 +00:00
jsegarra added 3 commits 2024-03-27 14:21:01 +00:00
jsegarra added 1 commit 2024-03-28 11:01:54 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
65a6174e2b
refs #6321 updates
jsegarra added 1 commit 2024-03-28 23:50:54 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
2cff160c6a
Merge branch 'dev' into 6321_negative_tickets
jsegarra added 2 commits 2024-04-01 12:00:25 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
d225821a41
refs #6321 test: itemLack
jsegarra added 3 commits 2024-04-01 14:13:40 +00:00
jsegarra requested review from jgallego 2024-04-01 15:05:45 +00:00
jsegarra changed title from WIP: #6321 - Negative tickets to #6321 - Negative tickets 2024-04-01 15:05:48 +00:00
jsegarra requested review from alexm 2024-04-01 15:05:58 +00:00
jgallego requested changes 2024-04-02 05:36:17 +00:00
@ -0,0 +2,4 @@
DELIMITER $$
$$
CREATE DEFINER=`root`@`localhost` PROCEDURE `vn`.`item_getLack`(
Owner

hay que usar el propio archivo del procedimiento db/routines/vn/procedures/item_getLack.sql

hay que usar el propio archivo del procedimiento db/routines/vn/procedures/item_getLack.sql
Author
Member

Se han colado las diferentes pruebas que hice antes de decirme que me base en "Monitor de ventas"

Se han colado las diferentes pruebas que hice antes de decirme que me base en "Monitor de ventas"
jsegarra marked this conversation as resolved
@ -0,0 +1,80 @@
DROP PROCEDURE IF EXISTS vn.item_getLack;
Owner

repetido

repetido
Author
Member

Se han colado las diferentes pruebas que hice antes de decirme que me base en "Monitor de ventas"

Se han colado las diferentes pruebas que hice antes de decirme que me base en "Monitor de ventas"
jsegarra marked this conversation as resolved
@ -0,0 +1,38 @@
SELECT
Owner

qual es el objetivo de este fichero?

qual es el objetivo de este fichero?
Author
Member

Se han colado las diferentes pruebas que hice antes de decirme que me base en "Monitor de ventas"

Se han colado las diferentes pruebas que hice antes de decirme que me base en "Monitor de ventas"
jsegarra marked this conversation as resolved
@ -0,0 +93,4 @@
}) ?? {};
const stmts = [];
stmts.push(`SET @_optimizer_search_depth = @@optimizer_search_depth`);
Owner

si se va a migrar el proc tal cual confirmar con Juan pero yo no lo moveria que se llame a item_getLack de la BBDD directamente

si se va a migrar el proc tal cual confirmar con Juan pero yo no lo moveria que se llame a item_getLack de la BBDD directamente
Author
Member

Lo anoto para hablar con el

Lo anoto para hablar con el
Author
Member

La idea de hacer este movimiento, es poder aplicar filtros a la consulta. Esto justo se esta haciendo en "Monitor de ventas". Hay mucho código SQL definido en el método para poder aplicar los filtros que vienen por parámetros

La idea de hacer este movimiento, es poder aplicar filtros a la consulta. Esto justo se esta haciendo en "Monitor de ventas". Hay mucho código SQL definido en el método para poder aplicar los filtros que vienen por parámetros
Author
Member

Tras hablar con Juan, consideramos mejor enfoque modificar el procedimiento añadiendo tantos argumentos como filtros tengamos

Tras hablar con Juan, consideramos mejor enfoque modificar el procedimiento añadiendo tantos argumentos como filtros tengamos
jsegarra marked this conversation as resolved
@ -0,0 +2,4 @@
const ParameterizedSQL = require('loopback-connector').ParameterizedSQL;
module.exports = Self => {
Self.remoteMethod('itemLackDetail', {
description: 'Download a ticket delivery note document',
Owner

esta descripcion corresponde ?

esta descripcion corresponde ?
jsegarra marked this conversation as resolved
@ -0,0 +6,4 @@
accessType: 'READ',
accepts: [
{
arg: 'id',
Owner

si estamos en la seccion ticket, yo el argumento lo llamaria itemFk, porque a mitad codigo, id puede dar confusion a que es el id de la entidad, en este caso ticket

si estamos en la seccion ticket, yo el argumento lo llamaria itemFk, porque a mitad codigo, id puede dar confusion a que es el id de la entidad, en este caso ticket
jsegarra marked this conversation as resolved
@ -0,0 +28,4 @@
myOptions.transaction = tx;
}
const conn = Self.dataSource.connector;
const stmts = ['SELECT * FROM vn.negativeOrigin'];
Owner

esto no puede ser un find de loopback?

esto no puede ser un find de loopback?
Author
Member

La tabla negativeOrigin no está publicada en el model-config.json.
Pensé lo mismo, y supuse que si no se hizo fue por un motivo

La tabla negativeOrigin no está publicada en el model-config.json. Pensé lo mismo, y supuse que si no se hizo fue por un motivo
Owner

porque serás el primero que la necesita, publicala.

porque serás el primero que la necesita, publicala.
Author
Member

Okey, tomo nota para ahcer las modificaciones necesarias

Okey, tomo nota para ahcer las modificaciones necesarias
jsegarra marked this conversation as resolved
@ -0,0 +32,4 @@
tx = await Self.beginTransaction({});
myOptions.transaction = tx;
}
// const conn = Self.dataSource.connector;
Owner

quitar comentarios

quitar comentarios
Author
Member

Resuelto

Resuelto
jsegarra marked this conversation as resolved
@ -0,0 +55,4 @@
SELECT @vNewTicket vNewTicket;`,
[tid], myOptions);
if (vNewTicket === 0) continue;
Owner

en que caso devuelve un ticket = 0?

en que caso devuelve un ticket = 0?
Author
Member

Diría que en ningún caso. Pero esta condición estaba en access y la puse

Diría que en ningún caso. Pero esta condición estaba en access y la puse
jsegarra added 1 commit 2024-04-02 06:04:48 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
d08535ac18
refs #6321 minor changes
jsegarra requested review from jgallego 2024-04-02 13:03:33 +00:00
jsegarra added 1 commit 2024-04-03 07:49:46 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
c4f8734d44
refs #6321 fix: param
jsegarra added 2 commits 2024-04-08 09:34:18 +00:00
jsegarra added 1 commit 2024-04-08 10:16:51 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
3dd162b683
refs #6321 test: fix
jsegarra added 1 commit 2024-04-08 10:18:17 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
e0712645a2
refs #6321 test: fix
jgallego requested changes 2024-04-08 12:50:06 +00:00
@ -0,0 +35,4 @@
try {
const ticketsIds = tickets.map(({ticketFk}, index) => ticketFk);
const ticketsCount = await Self.rawSql(`
Owner

si el objetivo es contar tickets te sobraria la tabla sale,
si el objetivo es contar sales te sobraria la tabla ticket

si el objetivo es contar tickets te sobraria la tabla sale, si el objetivo es contar sales te sobraria la tabla ticket
jsegarra added 1 commit 2024-04-16 05:13:38 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
41f0b6aa93
Merge branch 'dev' into 6321_negative_tickets
jsegarra added 5 commits 2024-04-29 09:56:38 +00:00
jsegarra added 1 commit 2024-05-02 11:16:06 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
63d07cb082
Merge branch 'dev' into 6321_negative_tickets
jsegarra added 1 commit 2024-05-02 13:19:13 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
7caea44427
feat(salix): refs #6321 #6321 getSimilar
jsegarra added 2 commits 2024-05-08 10:31:01 +00:00
jorgep added 1 commit 2024-05-13 11:26:15 +00:00
jsegarra added 2 commits 2024-05-13 11:51:24 +00:00
jsegarra added 2 commits 2024-05-15 06:55:54 +00:00
jsegarra added 1 commit 2024-05-15 14:26:10 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
70f245fd2d
Merge branch 'dev' into 6321_negative_tickets
jsegarra added 1 commit 2024-05-24 11:52:56 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
8366cfa348
Merge branch 'dev' into 6321_negative_tickets
jsegarra added 2 commits 2024-05-28 10:50:44 +00:00
jsegarra added 1 commit 2024-05-31 13:09:33 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
7468f87808
feat(salix): refs #6321 #6321 improve split mehtod
jsegarra added 1 commit 2024-06-04 14:13:44 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
149aeac54e
Merge branch 'dev' into 6321_negative_tickets
jsegarra added 2 commits 2024-06-11 12:05:21 +00:00
jsegarra added 4 commits 2024-06-17 11:06:59 +00:00
jsegarra added 1 commit 2024-06-19 22:06:06 +00:00
alexm requested changes 2024-06-25 11:41:49 +00:00
alexm left a comment
Member

Cuando este para revisar ya me pones

Cuando este para revisar ya me pones
@ -112,3 +112,3 @@
runner.addReporter(new JunitReporter.JUnitXmlReporter());
}
if (opts.ci)
if (opts.ci || opts.debug)
Member

Quitar supongo

Quitar supongo
@ -0,0 +4,4 @@
module.exports = Self => {
Self.remoteMethod('itemLack', {
description: 'Download a ticket delivery note document',
Member

Cambiar descripccion

Cambiar descripccion
jsegarra added 5 commits 2024-07-03 21:06:33 +00:00
jsegarra added 2 commits 2024-07-18 20:04:37 +00:00
jsegarra added 4 commits 2024-07-19 22:37:22 +00:00
jsegarra added 2 commits 2024-07-22 14:47:13 +00:00
jsegarra added 2 commits 2024-07-23 21:09:58 +00:00
jsegarra added 1 commit 2024-09-11 06:50:44 +00:00
jsegarra added 5 commits 2024-09-16 10:15:07 +00:00
jsegarra added 7 commits 2024-09-24 20:14:12 +00:00
Some checks failed
gitea/salix/pipeline/pr-dev There was a failure building this commit
Required
Details
This pull request has changes conflicting with the target branch.
  • db/dump/fixtures.before.sql
  • db/routines/vn/procedures/item_getSimilar.sql
  • loopback/locale/es.json
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
4 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#1945
No description provided.