1781-zoneHoliday #994
No reviewers
Labels
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix#994
Loading…
Reference in New Issue
No description provided.
Delete Branch "1781-zoneHoliday"
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?
Antes de subir este PR se debé eliminar db/changes/10470-family/00-zoneHoliday.sql. Ya que esos cambios los he subido desde vn-database.
@ -0,0 +1,2 @@
INSERT INTO `salix`.`ACL`(`model`, `property`, `accessType`, `permission`, `principalType`, `principalId`)
VALUES('ZoneExclusionGeo', '*', '*', 'ALLOW', 'ROLE', 'employee');
check ACL for this model
@ -0,0 +1,402 @@
CREATE TABLE `vn`.`zoneExclusionGeo` (
remove this
@ -0,0 +1,48 @@
module.exports = Self => {
Self.remoteMethod('exclusionGeo', {
description: 'Exclude a zoneGeo for a zone',
from
@ -0,0 +29,4 @@
verb: 'POST'
}
});
where are my tests and my transaction?
@ -78,0 +78,4 @@
this.geoExclusions = {};
let geoExclusions = value.geoExclusions;
if (geoExclusions) {
is this testes?
@ -7,2 +8,3 @@
this.vnWeekDays = vnWeekDays;
this.editMode = 'include';
this.editMode = 'exclude';
this.exclusions;
this is undefined anyways
@ -21,2 +24,4 @@
}
get exclusionsGeoPath() {
return `Zones/exclusionGeo`;
no need for a getter if you don't use a setter in this case.
@ -0,0 +1,4 @@
INSERT INTO `salix`.`ACL`(`model`, `property`, `accessType`, `permission`, `principalType`, `principalId`)
Puedes mover este sql a la carpeta del sprint "junio" el de la familia aparece ya en tests y no se si va a dar problemas.
@ -22,3 +2,1 @@
"Cannot change the payment method if no salesperson": "No se puede cambiar la forma de pago si no hay comercial asignado",
"can't be blank": "El campo no puede estar vacío",
"Observation type must be unique": "El tipo de observación no puede repetirse",
"Name cannot be blank": "Name cannot be blank",
Estas seguro de querer destruir tantas traducciones?
@ -0,0 +17,4 @@
},
{
arg: 'geoIds',
type: 'any',
geoIds a mi entender son Identificadores de geo por ejemplo: [1, 2, 3, 4]
pero poner un nombre que reprensentaria una coleccion de ids y darle de tipo "any" no es buena practica resultando en un manejo de datos confuso.
refactoriza los argumentos para que reciba lo que necesita que aparentemente son ids a secas en una array.
@ -0,0 +27,4 @@
try {
const options = {transaction: tx};
const geoIds = [{id: 1}, {id: 2}];
const geoIds = [1, 2]
@ -0,0 +26,4 @@
try {
const options = {transaction: tx};
const zoneId = 2;
const geoIds = [{id: 1}, {id: 2}];
mas de lo mismo
@ -0,0 +11,4 @@
},
{
arg: 'geoIds',
type: 'any',
mala practica
LGTM