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

refs #5881 createRolAndGrantAcl #1715

Merged
pablone merged 5 commits from 5881-createRolClaim into dev 2023-08-24 11:49:45 +00:00
Conversation 0 Commits 5 Files Changed 1 +32
pablone commented 2023-08-24 06:13:18 +00:00
Member
Copy Link
No description provided.
pablone added 1 commit 2023-08-24 06:13:20 +00:00
gitea/salix/pipeline/head This commit looks good Details
470f511b30 refs #5881 createRolAndGrantAcl
pablone changed title from WIP: refs #5881 createRolAndGrantAcl to refs #5881 createRolAndGrantAcl 2023-08-24 06:21:45 +00:00
pablone requested review from alexm 2023-08-24 06:21:51 +00:00
pablone added 1 commit 2023-08-24 06:22:06 +00:00
gitea/salix/pipeline/head This commit looks good Details
fb9b249c7b Merge branch 'dev' into 5881-createRolClaim
alexm reviewed 2023-08-24 06:53:49 +00:00
db/changes/233601/00-createClaimReader.sql Outdated
@ -0,0 +1,33 @@
INSERT INTO `account`.`role` (`id`, `name`, `description`, `hasLogin`)
VALUES (122, 'claimViewer','Trabajadores que consulta las reclamaciones ',1);
alexm commented 2023-08-24 06:53:49 +00:00
Member
Copy Link

LLeva el id

LLeva el id
pablone marked this conversation as resolved
alexm requested changes 2023-08-24 06:54:36 +00:00
db/changes/233601/00-createClaimReader.sql Outdated
@ -0,0 +2,4 @@
VALUES (122, 'claimViewer','Trabajadores que consulta las reclamaciones ',1);
INSERT INTO `account`.`roleInherit` (`role`,`inheritsFrom`)
VALUES (35, 122);
alexm commented 2023-08-24 06:54:34 +00:00
Member
Copy Link

Mira si pots traure el id dinamicament, sino si quan pujes, algu ja ha fet un rol nou te fallara el insert id 122

Mira si pots traure el id dinamicament, sino si quan pujes, algu ja ha fet un rol nou te fallara el insert id 122
pablone marked this conversation as resolved
pablone added 1 commit 2023-08-24 08:04:39 +00:00
gitea/salix/pipeline/head This commit looks good Details
3694fcbbfb refs #5881 fixCreateRol
pablone requested review from alexm 2023-08-24 08:14:57 +00:00
alexm reviewed 2023-08-24 08:45:02 +00:00
db/changes/233601/00-createClaimReader.sql
@ -0,0 +1,33 @@
INSERT INTO `account`.`role` (`id`, `name`, `description`, `hasLogin`)
VALUES ('claimViewer','Trabajadores que consulta las reclamaciones ',1);
INSERT INTO `account`.`roleInherit` (`role`,`inheritsFrom`)
alexm commented 2023-08-24 08:45:02 +00:00
Member
Copy Link

Molt bona!

Molt bona!
pablone marked this conversation as resolved
alexm requested changes 2023-08-24 08:46:44 +00:00
db/changes/233601/00-createClaimReader.sql
@ -0,0 +21,4 @@
'getSummary'
);
INSERT INTO `salix`.`ACL` (`model`,`property`,`accessType`,`permission`,`principalType`,`principalid`)
alexm commented 2023-08-24 08:46:40 +00:00
Member
Copy Link

Asi igual te has rallat un poc no??
Si haveres ficat els 4 insert jo crec que quedava mes clar i te lleves el select no?

Asi igual te has rallat un poc no?? Si haveres ficat els 4 insert jo crec que quedava mes clar i te lleves el select no?
pablone marked this conversation as resolved
alexm added 1 commit 2023-08-24 08:46:58 +00:00
gitea/salix/pipeline/head There was a failure building this commit Details
c997ad6ccb Merge branch 'dev' into 5881-createRolClaim
pablone added 1 commit 2023-08-24 09:31:20 +00:00
gitea/salix/pipeline/head This commit looks good Details
c5006b6be6 refs #5881 insertAcl
pablone requested review from alexm 2023-08-24 09:32:03 +00:00
alexm approved these changes 2023-08-24 11:45:09 +00:00
pablone merged commit 671e34da9a into dev 2023-08-24 11:49:45 +00:00
pablone deleted branch 5881-createRolClaim 2023-08-24 11:49:45 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
alexm
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#1715
No description provided.
Delete Branch "5881-createRolClaim"

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?