3247-refactor(ticket): add more text in the error when change zone #841

Merged
carlosjr merged 2 commits from 3247-ticket_basic-data into dev 2022-01-10 10:41:27 +00:00
Member
No description provided.
alexm added the
CR / Tests passed
Simple
labels 2022-01-07 07:31:09 +00:00
alexm added 1 commit 2022-01-07 07:31:10 +00:00
carlosjr requested changes 2022-01-10 09:58:32 +00:00
@ -84,2 +84,2 @@
if (!zoneShipped || zoneShipped.zoneFk != args.zoneId)
throw new UserError(`You don't have privileges to change the zone`);
if (!zoneShipped || zoneShipped.zoneFk != args.zoneId) {
throw new UserError(`You don't have privileges to change the zone
Contributor

there are some typos on the error message, here's the correction:

'You don't have privileges to change the zone or for these parameters there are more than one shipping options, talk to agencies'

Also try extracting long arguments to constants.

there are some typos on the error message, here's the correction: 'You don't have privileges to change the zone or for these parameters there are more than one shipping options, talk to agencies' Also try extracting long arguments to constants.
alexm added 1 commit 2022-01-10 10:38:13 +00:00
gitea/salix/pipeline/head This commit looks good Details
5462af91ee
extract to variable
carlosjr approved these changes 2022-01-10 10:41:11 +00:00
carlosjr left a comment
Contributor

LGTM

LGTM
carlosjr merged commit 612311b23f into dev 2022-01-10 10:41:27 +00:00
carlosjr deleted branch 3247-ticket_basic-data 2022-01-10 10:41:28 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
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#841
No description provided.