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

feat: refs #7905 getBuysCsv #2900

Merged
guillermo merged 10 commits from 7905-getBuysCsv into dev 2024-09-05 09:44:39 +00:00
Conversation 1 Commits 10 Files Changed 4 +88
guillermo commented 2024-08-29 10:19:30 +00:00
Member
Copy Link
No description provided.
guillermo added 2 commits 2024-08-29 10:19:31 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
72b4607d54 feat: refs #7905 Added new method getBuysCsv
guillermo changed title from 7905-getBuysCsv to feat: refs #7905 getBuysCsv 2024-08-29 10:20:54 +00:00
guillermo added 1 commit 2024-08-29 10:22:33 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
e50c67c305 feat: refs #7905 Added new method getBuysCsv
guillermo requested review from jgallego 2024-08-29 10:28:54 +00:00
guillermo changed title from feat: refs #7905 getBuysCsv to WIP: feat: refs #7905 getBuysCsv 2024-08-29 10:36:22 +00:00
jgallego requested changes 2024-08-29 13:04:05 +00:00
Dismissed
modules/entry/back/methods/entry/getBuysCsv.js
@ -0,0 +1,75 @@
const UserError = require('vn-loopback/util/user-error');
jgallego commented 2024-08-29 13:04:00 +00:00
Owner
Copy Link

@jsegarra se quiere que los proveedores puedan tener sus buys. Ya lo hicimos via json, ahora lo quieren via csv, pero con este enfoque duplicamos codigo, hay forma de tenerlo unificado o por parametros ?

@jsegarra se quiere que los proveedores puedan tener sus buys. Ya lo hicimos via json, ahora lo quieren via csv, pero con este enfoque duplicamos codigo, hay forma de tenerlo unificado o por parametros ?
jsegarra commented 2024-08-29 18:22:38 +00:00
Member
Copy Link

Si, esto ya lo he visto pero tengo que buscar el commit
El ejemplo que tengo en la cabeza es cuando se planteo la necesidad de que printReport, según la cabecera generase un pdf o un html

Si, esto ya lo he visto pero tengo que buscar el commit El ejemplo que tengo en la cabeza es cuando se planteo la necesidad de que printReport, según la cabecera generase un pdf o un html
jsegarra commented 2024-08-29 18:22:53 +00:00
Member
Copy Link

Creo que la cabecera que buscamos es la de accept

Creo que la cabecera que buscamos es la de accept
jsegarra commented 2024-08-29 18:27:29 +00:00
Member
Copy Link

El commit
bb96125bbd

El commit https://gitea.verdnatura.es/verdnatura/salix/commit/bb96125bbda28164046ce7148aee6ccc934f9114
guillermo commented 2024-08-30 04:45:14 +00:00
Author
Member
Copy Link

#2900 (comment)

https://gitea.verdnatura.es/verdnatura/salix/pulls/2900#issuecomment-61247
guillermo marked this conversation as resolved
jgallego requested review from jsegarra 2024-08-29 13:04:11 +00:00
jsegarra reviewed 2024-08-29 18:30:32 +00:00
modules/entry/back/methods/entry/getBuysCsv.js
@ -0,0 +13,4 @@
http: {source: 'path'}
}
],
returns: [
jsegarra commented 2024-08-29 18:30:32 +00:00
Member
Copy Link

El problema que veo en mi propuesta es con respecto al return

El problema que veo en mi propuesta es con respecto al return
jsegarra commented 2024-08-29 18:36:47 +00:00
Member
Copy Link

Aunqaue puedes hacer que cuando tengas los datos o el stream, modificar ctx mediante
ctx.res.setHeader('Content-Type', 'text/csv'); ctx.res.setHeader('Content-Disposition', 'attachment; filename="discounted_price.csv"'); o ctx.res.setHeader('Content-Type', 'application/json');

Aunqaue puedes hacer que cuando tengas los datos o el stream, modificar ctx mediante `ctx.res.setHeader('Content-Type', 'text/csv'); ctx.res.setHeader('Content-Disposition', 'attachment; filename="discounted_price.csv"'); ` o ` ctx.res.setHeader('Content-Type', 'application/json');`
guillermo commented 2024-08-30 04:44:37 +00:00
Author
Member
Copy Link

Ya me he pegado de "ostias" para intentar tenerlo todo en un método, al final lo separé por el return como bien has dicho.

El segundo problema es que al retornar un JSON, claro el JSON puede tener objetos anidados, cosa que el CSV no, ya que es plano.

Por eso también obté por hacer la query directamente, ya que, a parte de lo mencionado, al utilizan loopback te crea objetos metadata en la variable que luego hay que eliminar y no se...

Ya me he pegado de "ostias" para intentar tenerlo todo en un método, al final lo separé por el return como bien has dicho. El segundo problema es que al retornar un JSON, claro el JSON puede tener objetos anidados, cosa que el CSV no, ya que es plano. Por eso también obté por hacer la query directamente, ya que, a parte de lo mencionado, al utilizan loopback te crea objetos metadata en la variable que luego hay que eliminar y no se...
jsegarra commented 2024-08-30 05:34:39 +00:00
Member
Copy Link

En ese caso eztraeria el core de la función getbuys a una función aparte y haría llamadas a esta nueva
Tipo lo que se hace en dms y docuware

En ese caso eztraeria el core de la función getbuys a una función aparte y haría llamadas a esta nueva Tipo lo que se hace en dms y docuware
jsegarra commented 2024-08-30 05:57:33 +00:00
Member
Copy Link

Ya te sigo, por lo que veo, cuando se usa toCSV no hay relaciones/includes
Sin embargo, modules/invoiceOut/back/methods/invoiceOut/negativeBasesCsv.js, obtiene datos y luego hace un map de los resultados
A lo mejor podriamos invertir tiempo para que dato un objeto(label:path), montar un csv

csv= {
   nombre: 'name',
   colorItem: 'item.color'
}
Ya te sigo, por lo que veo, cuando se usa toCSV no hay relaciones/includes Sin embargo, modules/invoiceOut/back/methods/invoiceOut/negativeBasesCsv.js, obtiene datos y luego hace un map de los resultados A lo mejor podriamos invertir tiempo para que dato un objeto(label:path), montar un csv ``` csv= { nombre: 'name', colorItem: 'item.color' } ```
jgallego commented 2024-09-02 06:11:06 +00:00
Owner
Copy Link

El tema de que el CSV no tenga objetos anidados No hay que darle una solución técnica sino hablar con el proveedor ya que implica que tenga que recuperar los datos de forma distinta

El tema de que el CSV no tenga objetos anidados No hay que darle una solución técnica sino hablar con el proveedor ya que implica que tenga que recuperar los datos de forma distinta
guillermo marked this conversation as resolved
jsegarra added 1 commit 2024-08-30 07:42:55 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
0f4bc3f83b feat(salix): refs #7905 #7905 use getBuys toCSV flattened
jsegarra added 1 commit 2024-08-30 21:38:56 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
8aa4427693 Merge branch 'dev' into 7905-getBuysCsv
jsegarra requested review from jgallego 2024-08-30 21:38:58 +00:00
jsegarra commented 2024-08-30 21:39:25 +00:00
Member
Copy Link

@jgallego Hemos hecho una propuesta para usar getBuys desde getBuysCsv

@jgallego Hemos hecho una propuesta para usar getBuys desde getBuysCsv
jgallego approved these changes 2024-09-05 06:29:28 +00:00
guillermo changed title from WIP: feat: refs #7905 getBuysCsv to feat: refs #7905 getBuysCsv 2024-09-05 06:33:10 +00:00
jsegarra added 1 commit 2024-09-05 07:13:38 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
d49707828b fix(salix): refs #7905 #7905 use right fn to flatten data
jsegarra approved these changes 2024-09-05 07:16:30 +00:00
guillermo added 1 commit 2024-09-05 07:18:39 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
2b465d0d23 Merge branch 'dev' into 7905-getBuysCsv
guillermo added 2 commits 2024-09-05 07:21:05 +00:00
guillermo added 1 commit 2024-09-05 07:26:29 +00:00
gitea/salix/pipeline/pr-dev This commit looks good Details
d0eab064b7 Merge branch 'dev' into 7905-getBuysCsv
guillermo merged commit 620ce5d45e into dev 2024-09-05 09:44:39 +00:00
guillermo deleted branch 7905-getBuysCsv 2024-09-05 09:44:39 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
jgallego
jsegarra
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
3 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#2900
No description provided.
Delete Branch "7905-getBuysCsv"

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?