#6694 Filter postcode by multiple fields #1939

Merged
jsegarra merged 7 commits from 6694_postcode_location_filter into dev 2024-01-22 07:22:59 +00:00
Member
No description provided.
jsegarra added 1 commit 2024-01-17 07:34:41 +00:00
gitea/salix/pipeline/head This commit looks good Details
30b505aa8f
refs #6694 feat: add filter for postcodes
jsegarra changed title from WIP: #6694 Filter postcode by multiple fields to #6694 Filter postcode by multiple fields 2024-01-17 08:35:30 +00:00
jsegarra requested review from alexm 2024-01-17 08:35:34 +00:00
alexm requested changes 2024-01-17 12:52:02 +00:00
alexm left a comment
Member

Y si puedes hazle algun test

Y si puedes hazle algun test
@ -1,6 +1,38 @@
let UserError = require('vn-loopback/util/user-error');
module.exports = Self => {
Self.remoteMethod('filter', {
Member

La ruta iria en su propio archivo.

La ruta iria en su propio archivo.
Author
Member

Quieres decir que haga un archivo llamado "filter"(pej) para ese método?

Quieres decir que haga un archivo llamado "filter"(pej) para ese método?
Member

Exacto

Exacto
Author
Member

Corregido 71f387f626

Corregido 71f387f626add3952755e0c83b6ead88bd435941
jsegarra marked this conversation as resolved
@ -4,0 +16,4 @@
});
Self.filter = async(ctx, filter, options) => {
const {Postcode} = Self.app.models;
let {value} = ctx.where;
Member

Si no se modifica el valor serian const
Y veo raro lo de sacar el valor del where del ctx

Si no se modifica el valor serian const Y veo raro lo de sacar el valor del where del ctx
Author
Member

Cierto, es const

Cierto, es const
Author
Member

He usado como ejemplo el apartado de Claim.development del front.

si quieres revisamos como debería ser para atacar a la propiedad filter en vez del where

He usado como ejemplo el apartado de Claim.development del front. si quieres revisamos como debería ser para atacar a la propiedad filter en vez del where
Author
Member

Corregido lo de const 92a3e841ed

Corregido lo de const 92a3e841ed3fe6ec34ea6ff44339808bd6a71e30
jsegarra marked this conversation as resolved
@ -4,0 +29,4 @@
or t.name like '%${value}%'
or p.name like '%${value}%'
or c.country like '%${value}%'
LIMIT ${limit}
Member

Aqui te podria inyectar codigo.
Mejor usar:

LIMIT ?`) , [limit]

Aqui te podria inyectar codigo. Mejor usar: LIMIT ?`) , [limit]
Author
Member

Cierto, he usado la misma lógica del like, pero en este caso si que estaría mal.
Gracias

Cierto, he usado la misma lógica del like, pero en este caso si que estaría mal. Gracias
Member

Cierto, en el like creo que tambien seria incorrecto hacerlo

Cierto, en el like creo que tambien seria incorrecto hacerlo
Author
Member

Corregido 38c93e8c76

Corregido 38c93e8c769ad79387b6a0e30bb92e036a7ffb7a
jsegarra marked this conversation as resolved
jsegarra added 3 commits 2024-01-17 13:17:22 +00:00
jsegarra changed title from #6694 Filter postcode by multiple fields to WIP: #6694 Filter postcode by multiple fields 2024-01-17 13:17:47 +00:00
Author
Member

Y si puedes hazle algun test

Añadido 38c93e8c76

> Y si puedes hazle algun test Añadido 38c93e8c769ad79387b6a0e30bb92e036a7ffb7a
jsegarra added 2 commits 2024-01-17 13:59:43 +00:00
jsegarra requested review from alexm 2024-01-17 14:00:23 +00:00
jsegarra changed title from WIP: #6694 Filter postcode by multiple fields to #6694 Filter postcode by multiple fields 2024-01-17 14:00:28 +00:00
alexm added 1 commit 2024-01-22 06:41:36 +00:00
gitea/salix/pipeline/head This commit looks good Details
51f7fcf439
Merge branch 'dev' into 6694_postcode_location_filter
alexm approved these changes 2024-01-22 07:13:01 +00:00
jsegarra merged commit 0aecda6776 into dev 2024-01-22 07:22:59 +00:00
jsegarra deleted branch 6694_postcode_location_filter 2024-01-22 07:22:59 +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#1939
No description provided.