refs #6085 aclMail back #1869
Labels
No Milestone
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix#1869
Loading…
Reference in New Issue
No description provided.
Delete Branch "6085-ACLsMail"
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?
@ -0,0 +1,7 @@
-- Definición de la tabla mailAliasACL
CREATE TABLE `account`.`mailAliasACL` (
CREATE OR REPLACE
Revisa el back y carregat el front tot lo demes guay
@ -0,0 +1,70 @@
const UserError = require('vn-loopback/util/user-error');
module.exports = Self => {
require('../methods/notification/getList')(Self);
mmmm
@ -0,0 +11,4 @@
await checkModifyPermission(ctx);
});
async function checkModifyPermission(ctx) {
Esta part mirala be pq soles has de comprovar el userId del que esta fent el canvi const userId = ctx.options.accessToken.userId;
No se usa ese fichero, se usa account, solucionado
@ -0,0 +1,33 @@
import ngModule from '../../../module';
El front carregatelo
WIP: refs #6085 aclMail backto refs #6085 aclMail back@ -0,0 +1,8 @@
-- Definición de la tabla mailAliasACL
Cambiar a nueva versión
@ -68,3 +68,4 @@ Load more results: Cargar más resultados
Send cau: Enviar cau
By sending this ticket, all the data related to the error, the section, the user, etc., are already sent.: Al enviar este cau ya se envían todos los datos relacionados con el error, la sección, el usuario, etc
ExplainReason: Explique el motivo por el que no deberia aparecer este fallo
You already have the mailAlias: Ya tienes este mail
este alias de correo
@ -331,2 +331,3 @@
"Cannot past travels with entries": "No se pueden pasar envíos con entradas",
"It was not able to remove the next expeditions:": "No se pudo eliminar las siguientes expediciones: {{expeditions}}"
"It was not able to remove the next expeditions:": "No se pudo eliminar las siguientes expediciones: {{expeditions}}",
"You already have the mailAlias": "You already have the mailAlias",
Habria que traducirlo aqui
@ -44,1 +47,4 @@
roleFk: {
inq: roles.map(role => role.roleId),
},
}
Estic dubtant de si igual valdria la pena possar tb mailAliasFk: mailAliasFk
Aixina despres no cal buscarlo concretament.
Este enfoque se pareix al de notificactions (que es com te vaig dir) pero igual en este cas com no es crida des de altre lloc per a demanar el llistat igual no val la pena fer-ho igual
@ -0,0 +1,55 @@
const UserError = require('vn-loopback/util/user-error');
Pq el codi esta duplicat en modules/account/back/models/mail-alias-acl.js i modules/account/back/models/mail-alias-account.js ?
@ -33,3 +29,1 @@
const user = await models.VnUser.findById(userId, {fields: ['hasGrant']});
if (!user.hasGrant)
throw new UserError(`You don't have grant privilege`);
if (typeof options == 'object')
options no esta definida en ningún sitio
@ -52,3 +48,1 @@
return true;
};
console.log(availableMailAlias);
Se ha quedado un console log
@ -29,3 +26,2 @@
const canEditAlias = await models.ACL.checkAccessAcl(accessToken, 'MailAliasAccount', 'canEditAlias', 'WRITE');
if (canEditAlias) return true;
const myOptions = {};
myOptions no tiene utilidad
@ -41,3 +32,1 @@
scope: {
fields: ['mailAlias']
}
const availableMailAlias = await models.MailAliasAcl.find({
Un
findOne
sería mes adequat. I faría la inversa, primer traure els registres de MailAliasAcl i despres comprobar si el usuari te algún dells en findOneNew commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings