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

WIP: fix: refs #7917 fix methods, tests, acls #3085

Draft
carlossa wants to merge 9 commits from 7917-freelancerRoute into dev
pull from: 7917-freelancerRoute
merge into: verdnatura:dev
Conversation 0 Commits 9 Files Changed 10 +107 -57
carlossa commented 2024-10-08 10:19:25 +00:00
Member
Copy Link
No description provided.
carlossa added 1 commit 2024-10-08 10:19:25 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
273068e595 fix: refs #7917 fix methods, tests, acls
jgallego requested changes 2024-11-13 08:54:15 +00:00
db/versions/11292-tealChrysanthemum/00-firstScript.sql
@ -0,0 +1,6 @@
INSERT INTO salix.ACL (model, property, accessType, permission, principalType, principalId)
VALUES
('Route', 'getTickets', 'READ', 'ALLOW', 'ROLE', 'supplier'),
jgallego commented 2024-11-13 08:54:10 +00:00
Owner
Copy Link

confirmar con jbreso que rol tienen los autonomos que van a usar esto

confirmar con jbreso que rol tienen los autonomos que van a usar esto
carlossa commented 2024-11-19 10:41:48 +00:00
Author
Member
Copy Link

Lo van a ver usuarios delivery

Lo van a ver usuarios delivery
modules/route/back/methods/agency-term/specs/filter.spec.js Outdated
@ -30,6 +30,7 @@ describe('AgencyTerm filter()', () => {
it('should return results matching "search" searching by integer', async() => {
let ctx = {
args: {
ctx: {req: {accessToken: {userId: authUserId}}},
jgallego commented 2024-11-13 08:48:05 +00:00
Owner
Copy Link

posau comú en la par superior, en el beforeall hi ha exemples

posau comú en la par superior, en el beforeall hi ha exemples
modules/supplier/back/models/supplier.js
@ -163,2 +163,4 @@
throw new UserError('The social name has an invalid format');
});
Self.isSupplier = async(ctx, options = {}) => {
jgallego commented 2024-11-13 08:50:21 +00:00
Owner
Copy Link

aci es mes sencill que pases el userId directament

aci es mes sencill que pases el userId directament
carlossa marked this conversation as resolved
carlossa added 1 commit 2024-11-15 12:38:13 +00:00
carlossa added 1 commit 2024-11-18 07:29:48 +00:00
carlossa added 1 commit 2024-11-18 07:32:53 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
5e7dea94b1 fix: refs #7917 tests
carlossa added 1 commit 2024-11-18 07:47:11 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
bbe62ece6e fix: refs #7917 model
carlossa added 1 commit 2024-11-18 09:54:01 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
0c4985604b fix: refs #7917 test back
carlossa added 1 commit 2024-11-19 10:44:52 +00:00
carlossa added 1 commit 2024-11-19 11:34:12 +00:00
gitea/salix/pipeline/pr-dev There was a failure building this commit Details
839475a669 fix: refs #7917 fix tback
carlossa added 1 commit 2024-11-22 10:12:57 +00:00
Some checks failed
gitea/salix/pipeline/pr-dev There was a failure building this commit
Required
Details
This pull request is marked as a work in progress.
This branch is out-of-date with the base branch
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#3085
No description provided.
Delete Branch "7917-freelancerRoute"

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?