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

5097-item.summary_available #1312

Merged
vicent merged 16 commits from 5097-item.summary_available into dev 2023-02-17 08:54:23 +00:00
Conversation 0 Commits 16 Files Changed 19 +141 -68
vicent commented 2023-02-08 07:29:22 +00:00
Contributor
Copy Link
No description provided.
vicent added the
Missing tests
 label 2023-02-08 07:29:22 +00:00
vicent added 9 commits 2023-02-08 07:29:23 +00:00
gitea/salix/pipeline/head This commit looks good Details
eefafb3ac2 fix: available is cero
gitea/salix/pipeline/head There was a failure building this commit Details
caf1dcd690 feat: añadido almacén por defecto
gitea/salix/pipeline/head There was a failure building this commit Details
366748a136 fix: se la pasa la fecha shipped para calcular el visible y disponible
gitea/salix/pipeline/head There was a failure building this commit Details
19120ef102 feat: el descriptor coge el warehouseFk dependiendo de la sección
gitea/salix/pipeline/head There was a failure building this commit Details
ba5616055b feat: item.descriptor en summary coge el warehouse por defecto
gitea/salix/pipeline/head There was a failure building this commit Details
4f76465246 fix: no recarga cada vez
gitea/salix/pipeline/head There was a failure building this commit Details
6cb3930be0 feat: añadido icono info, que indica de que almacén se saca la información
jgallego requested changes 2023-02-08 07:36:47 +00:00
db/changes/230601/00-itemConfig_warehouseFk.sql Outdated
@ -0,0 +1,4 @@
ALTER TABLE `vn`.`itemConfig` ADD warehouseFk smallint(6) unsigned NULL;
UPDATE `vn`.`itemConfig`
jgallego commented 2023-02-08 07:30:31 +00:00
Owner
Copy Link

quitar el update de itemConfig, debe hacerse directamente en produccion cuando se suba

quitar el update de itemConfig, debe hacerse directamente en produccion cuando se suba
vicent commented 2023-02-08 08:40:43 +00:00
Author
Contributor
Copy Link

Ara ho cambie, pero pq no es pot en un chages.sql?

Ara ho cambie, pero pq no es pot en un chages.sql?
modules/item/back/methods/item/getSummary.js Outdated
@ -99,3 +99,3 @@
[summary.botanical] = res[2];
const userConfig = await models.UserConfig.getUserConfig(ctx, myOptions);
const itemConfig = await models.ItemConfig.findOne({where: {code: 'VNL'}}, myOptions);
jgallego commented 2023-02-08 07:30:44 +00:00
Owner
Copy Link

que fa esta linea?

que fa esta linea?
vicent commented 2023-02-08 08:39:29 +00:00
Author
Contributor
Copy Link

Agarra el unic registre de itemConfig, pero he vist q el where sobra. Ho modifique també.

Agarra el unic registre de itemConfig, pero he vist q el where sobra. Ho modifique també.
vicent added 1 commit 2023-02-08 08:41:41 +00:00
vicent requested review from jgallego 2023-02-08 08:41:55 +00:00
vicent added 1 commit 2023-02-16 06:15:12 +00:00
vicent added 1 commit 2023-02-16 06:33:29 +00:00
gitea/salix/pipeline/head This commit looks good Details
bb2eddc462 fix: test front
vicent added 2 commits 2023-02-16 08:44:59 +00:00
gitea/salix/pipeline/head This commit looks good Details
3dd870f7df fix: test e2e
vicent added
CR / Tests passed
 and removed
Missing tests
 labels 2023-02-16 08:48:02 +00:00
vicent added 1 commit 2023-02-16 08:51:05 +00:00
gitea/salix/pipeline/head This commit looks good Details
76f2030125 feat: add update sql
jgallego approved these changes 2023-02-17 06:58:22 +00:00
vicent added 6 commits 2023-02-17 08:48:01 +00:00
gitea/salix/pipeline/head There was a failure building this commit Details
b209ecb011 feat: create post route for debug
gitea/salix/pipeline/head This commit looks good Details
9ec75a5af3 Merge branch 'test' into 5094-application_post
gitea/salix/pipeline/head This commit looks good Details
80b3588b37 Merge pull request 'feat: create post route for debug' (!1315) from 5094-application_post into test 
Reviewed-on: #1315
Reviewed-by: Javi Gallego <jgallego@verdnatura.es>
Reviewed-by: Juan Ferrer <juan@verdnatura.es>
gitea/salix/pipeline/head This commit looks good Details
3031ae2511 Updated version
gitea/salix/pipeline/head This commit looks good Details
d25ebaec0b Merge branch 'dev' into 5097-item.summary_available
vicent scheduled this pull request to auto merge when all checks succeed 2023-02-17 08:49:07 +00:00
vicent merged commit af92aa60e9 into dev 2023-02-17 08:54:23 +00:00
vicent deleted branch 5097-item.summary_available 2023-02-17 08:57:39 +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#1312
No description provided.
Delete Branch "5097-item.summary_available"

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?