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

#7648 - Filter entries by days #2722

Merged
jsegarra merged 12 commits from 7648_myEntries_filter into dev 2024-07-15 09:22:35 +00:00
Conversation 1 Commits 12 Files Changed 6 +30 -15
jsegarra commented 2024-07-11 17:57:42 +00:00
Member
Copy Link
No description provided.
jsegarra added 3 commits 2024-07-11 17:57:42 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
1c5ab395e6 Merge branch 'dev' into 7648_myEntries_filter
jsegarra requested review from jgallego 2024-07-11 17:57:48 +00:00
jsegarra commented 2024-07-11 17:58:25 +00:00
Author
Member
Copy Link

He hecho el primer commit de manera atomica donde solo añado travels y entries y modifico los expected de los tests
Una vez eso, hago la funcionalidad

He hecho el primer commit de manera atomica donde solo añado travels y entries y modifico los expected de los tests Una vez eso, hago la funcionalidad
jgallego requested changes 2024-07-12 05:55:11 +00:00
Dismissed
db/dump/fixtures.after.sql Outdated
@ -318,0 +319,4 @@
SET `role`=131
WHERE id=1102;
UPDATE vn.travel
jgallego commented 2024-07-12 05:55:07 +00:00
Owner
Copy Link

esto porque va en las fixtures after i no en las fixtures normales modificando la linea de creacion de esos travels?

esto porque va en las fixtures after i no en las fixtures normales modificando la linea de creacion de esos travels?
jsegarra commented 2024-07-12 06:39:54 +00:00
Author
Member
Copy Link

Porque tocaba travel y entry, y me parecía mas seguro hacerlo en el after por si rompia cosas.
La verdad es que en el before no lo probé.

Porque tocaba travel y entry, y me parecía mas seguro hacerlo en el after por si rompia cosas. La verdad es que en el before no lo probé.
jsegarra commented 2024-07-12 12:37:54 +00:00
Author
Member
Copy Link

Añadir un travel 9 y las entradas asociadas

Añadir un travel 9 y las entradas asociadas
jsegarra commented 2024-07-12 20:41:53 +00:00
Author
Member
Copy Link

He tenido que añadir el travel 10 porque ya había un insert en fixture.after

He tenido que añadir el travel 10 porque ya había un insert en fixture.after
jsegarra marked this conversation as resolved
modules/entry/back/methods/entry/filter.js Outdated
@ -185,2 +191,3 @@
t.warehouseInFk,
w.name warehouseInName
w.name warehouseInName,
util.VN_CURDATE()
jgallego commented 2024-07-12 05:49:03 +00:00
Owner
Copy Link

Para qué necesitas esta línea?

Para qué necesitas esta línea?
jsegarra commented 2024-07-12 12:27:39 +00:00
Author
Member
Copy Link

Está eliminada

Está eliminada
jgallego marked this conversation as resolved
modules/entry/back/methods/entry/filter.js Outdated
@ -195,0 +203,4 @@
if (supplier) {
stmt.merge({
sql: `
AND t.shipped < util.VN_CURDATE() + INTERVAL ? + 1 DAY
jgallego commented 2024-07-12 05:53:00 +00:00
Owner
Copy Link

mira en otras secciones que tengamos parametros por defecto en los filter a ver como lo hemos gestionado los valores, para guardarlos, nose si @alexm tendra algun ejemplo de memoria

mira en otras secciones que tengamos parametros por defecto en los filter a ver como lo hemos gestionado los valores, para guardarlos, nose si @alexm tendra algun ejemplo de memoria
jsegarra commented 2024-07-12 06:52:39 +00:00
Author
Member
Copy Link

Okey, he hecho un cambio que a lo mejor era lo que hacia falta

Okey, he hecho un cambio que a lo mejor era lo que hacia falta
jsegarra commented 2024-07-12 12:27:55 +00:00
Author
Member
Copy Link

@jgallego es travel.shipped o landed?

@jgallego es travel.shipped o landed?
jgallego commented 2024-07-12 12:31:23 +00:00
Owner
Copy Link

shipped ellos quieren saber cuando sale

shipped ellos quieren saber cuando sale
jsegarra marked this conversation as resolved
jsegarra added 2 commits 2024-07-12 06:52:08 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
47cff3f894 feat(ssalix): refs #7648 remove select field
jsegarra requested review from jgallego 2024-07-12 06:52:43 +00:00
jgallego requested changes 2024-07-12 07:22:24 +00:00
Dismissed
modules/entry/back/methods/entry/filter.js Outdated
@ -2,6 +2,7 @@
const ParameterizedSQL = require('loopback-connector').ParameterizedSQL;
const buildFilter = require('vn-loopback/util/filter').buildFilter;
const mergeFilters = require('vn-loopback/util/filter').mergeFilters;
const DEFAULT_INTERVAL_DAYS = 3;
jgallego commented 2024-07-12 07:22:19 +00:00
Owner
Copy Link

el problema es seguimos teniendo un 3 en front i en back

el problema es seguimos teniendo un 3 en front i en back
jsegarra commented 2024-07-12 12:30:36 +00:00
Author
Member
Copy Link

Desde el front envias ese valor, pero si lo consumes desde el api tienes que darle un valor por defecto, es lo que has comentado mas arriba no?

Desde el front envias ese valor, pero si lo consumes desde el api tienes que darle un valor por defecto, es lo que has comentado mas arriba no?
jsegarra commented 2024-07-12 12:48:08 +00:00
Author
Member
Copy Link

Eliminar default y mover condicion al if

Eliminar default y mover condicion al if
jsegarra marked this conversation as resolved
jsegarra requested review from jgallego 2024-07-12 12:34:44 +00:00
jsegarra added 4 commits 2024-07-12 18:52:57 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
1d23554508 perf(ssalix): refs #7648 remove default value
jsegarra added 1 commit 2024-07-13 18:07:49 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
0ca4bffaf0 Perf: improve condition days filter
jgallego requested changes 2024-07-14 07:45:39 +00:00
Dismissed
db/dump/fixtures.after.sql Outdated
@ -315,4 +315,7 @@ INSERT INTO mysql.roles_mapping (`User`, `Host`, `Role`, `Admin_option`)
FROM mysql.roles_mapping
WHERE `User` LIKE @prefixedLike AND `Host` = @genRoleHost;
UPDATE account.`user`
jgallego commented 2024-07-14 07:45:38 +00:00
Owner
Copy Link

cambia esto tb donde se establece el rol del 1102 así queda unificado

cambia esto tb donde se establece el rol del 1102 así queda unificado
jsegarra marked this conversation as resolved
modules/entry/back/methods/entry/filter.js Outdated
@ -195,0 +199,4 @@
if (ctx.args.days) {
stmt.merge({
sql: `
AND t.shipped < util.VN_CURDATE() + INTERVAL ? + 1 DAY
jgallego commented 2024-07-14 07:44:36 +00:00
Owner
Copy Link

si pones <= no te puedes ahorar el interval + 1?

si pones <= no te puedes ahorar el interval + 1?
👀 1
jsegarra marked this conversation as resolved
jsegarra added 2 commits 2024-07-15 08:00:02 +00:00
jsegarra requested review from jgallego 2024-07-15 08:00:19 +00:00
jgallego approved these changes 2024-07-15 09:19:22 +00:00
jsegarra merged commit 87a3fb57fe into dev 2024-07-15 09:22:35 +00:00
jsegarra deleted branch 7648_myEntries_filter 2024-07-15 09:22:35 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
jgallego
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#2722
No description provided.
Delete Branch "7648_myEntries_filter"

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?