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

#4507 greuge.js/json now insert userFk @2h #1101

Merged
pau merged 14 commits from 4507-greuge-add-userFk into dev 2022-11-21 08:44:46 +00:00
Conversation 1 Commits 14 Files Changed 6 +36 -5
pau commented 2022-10-24 08:40:25 +00:00
Contributor
Copy Link
No description provided.
pau added the
Missing tests
 label 2022-10-24 08:40:25 +00:00
pau added 1 commit 2022-10-24 08:40:29 +00:00
gitea/salix/pipeline/head This commit looks good Details
30933220ac #4507 greuge.js/json now insert userFk @2h
pau commented 2022-10-24 08:44:35 +00:00
Author
Contributor
Copy Link

Me estan dando problemas los test de e2e, 95% seguro de que es cosa de mi PC, cuando llegue a casa eliminare los node_modules y relanzare npm install y gulp install

Me estan dando problemas los test de e2e, 95% seguro de que es cosa de mi PC, cuando llegue a casa eliminare los node_modules y relanzare npm install y gulp install
pau added
CR / Tests passed
 and removed
Missing tests
 labels 2022-11-04 07:34:56 +00:00
pau added 1 commit 2022-11-04 07:35:09 +00:00
gitea/salix/pipeline/head This commit looks good Details
7ff4c4d33c Merge branch 'dev' into 4507-greuge-add-userFk
joan added this to the v9.0.0 - November milestone 2022-11-04 08:24:41 +00:00
joan requested changes 2022-11-08 07:14:25 +00:00
modules/client/back/models/greuge.json Outdated
@ -35,2 +35,4 @@
"type": "number",
"required": true
},
"userFk": {
joan commented 2022-11-08 07:14:19 +00:00
Contributor
Copy Link

Necesitas listar la relación con worker para poder obtener su información en vez de la columna userFk

Necesitas listar la relación con worker para poder obtener su información en vez de la columna userFk
pau marked this conversation as resolved
modules/client/front/greuge/index/index.html Outdated
@ -37,6 +38,7 @@
<vn-tbody>
<vn-tr ng-repeat="greuge in greuges">
<vn-td shrink-datetime>{{::greuge.shipped | date:'dd/MM/yyyy HH:mm' }}</vn-td>
<vn-td>{{::greuge.userFk}}</vn-td>
joan commented 2022-11-08 07:13:21 +00:00
Contributor
Copy Link

No solemos mostrar el id de usuario, se debe acceder acceder a la relación y mostrar el nombre de usuario. Poner también con enlace que abre el descriptor del trabajador (worker-descriptor-popover)

No solemos mostrar el id de usuario, se debe acceder acceder a la relación y mostrar el nombre de usuario. Poner también con enlace que abre el descriptor del trabajador (worker-descriptor-popover)
pau marked this conversation as resolved
pau added 1 commit 2022-11-10 06:26:15 +00:00
gitea/salix/pipeline/head This commit looks good Details
eab9289408 Merge branch 'dev' into 4507-greuge-add-userFk
pau added 1 commit 2022-11-10 07:18:19 +00:00
gitea/salix/pipeline/head This commit looks good Details
1327f0ae9d Added requested changes, refs #4507 @1:20h
pau added 1 commit 2022-11-10 07:20:08 +00:00
gitea/salix/pipeline/head This commit looks good Details
c53e309071 Fix typo
pau added 2 commits 2022-11-10 07:36:40 +00:00
gitea/salix/pipeline/head This commit looks good Details
8b06243041 Fix link being the whole column
pau added 1 commit 2022-11-15 06:10:39 +00:00
gitea/salix/pipeline/head This commit looks good Details
29b299f36a Merge branch 'dev' into 4507-greuge-add-userFk
pau requested review from joan 2022-11-15 06:11:42 +00:00
pau added 1 commit 2022-11-16 06:08:27 +00:00
gitea/salix/pipeline/head This commit looks good Details
d7dc7c4329 Merge branch 'dev' into 4507-greuge-add-userFk
joan requested changes 2022-11-16 07:17:12 +00:00
db/changes/10502-november/00-greuge.userFK_userFk.sql Outdated
@ -0,0 +1 @@
ALTER TABLE vn.greuge CHANGE userFK userFk int(10) unsigned DEFAULT NULL NULL;
joan commented 2022-11-16 07:14:55 +00:00
Contributor
Copy Link

Faltan template strings. Mover a versión 10503

Faltan template strings. Mover a versión 10503
pau marked this conversation as resolved
pau added 1 commit 2022-11-16 07:28:26 +00:00
gitea/salix/pipeline/head This commit looks good Details
7b3a1bfb22 requested changes
pau added 1 commit 2022-11-16 07:28:49 +00:00
gitea/salix/pipeline/head This commit looks good Details
a3f56f1622 Merge branch 'dev' into 4507-greuge-add-userFk
pau added 1 commit 2022-11-16 13:34:59 +00:00
gitea/salix/pipeline/head This commit looks good Details
9257a02390 Merge branch 'dev' into 4507-greuge-add-userFk
pau requested review from joan 2022-11-16 13:37:23 +00:00
joan approved these changes 2022-11-18 13:46:23 +00:00
pau added 1 commit 2022-11-21 06:06:04 +00:00
gitea/salix/pipeline/head There was a failure building this commit Details
c320342476 Merge branch 'dev' into 4507-greuge-add-userFk
pau added 1 commit 2022-11-21 08:34:20 +00:00
gitea/salix/pipeline/head This commit looks good Details
cb0735d136 Merge branch 'dev' into 4507-greuge-add-userFk
pau merged commit 4302f0be02 into dev 2022-11-21 08:44:46 +00:00
pau deleted branch 4507-greuge-add-userFk 2022-11-21 08:44:46 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
joan
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
v9.0.0 - November
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#1101
No description provided.
Delete Branch "4507-greuge-add-userFk"

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?