#4074 useAcls #584
No reviewers
Labels
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#584
Loading…
Reference in New Issue
No description provided.
Delete Branch "4074-useAcls"
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?
WIP #4074 useAclsto #4074 useAcls@ -80,7 +80,6 @@ export default {
meta: {
title: 'basicData',
icon: 'vn:settings',
roles: ['salesPerson'],
No tiene sentido, solo puede entrar en InvoiceIn administrative.
@ -76,7 +76,6 @@ export default {
meta: {
title: 'basicData',
icon: 'vn:settings',
roles: ['salesPerson'],
No tiene sentido. Cualquier employee puede editarlo o verlo.
@ -26,3 +32,4 @@
const isAllowedToCreate = computed(() => {
if ($props.acls.length) return acl.hasAny($props.acls);
return role.hasAny($props.rolesAllowedToCreate);
Mantenemos retrocompatibilidad.
@ -53,8 +53,8 @@ describe('Login', () => {
});
it(`should get redirected to dashboard since employee can't create tickets`, () => {
cy.visit('/#/ticket/create', { failOnStatusCode: false });
Habían puesto que solo podía el role developer. Esto en Salix no estaba.
@ -9,3 +9,2 @@
import VnUserLink from 'src/components/ui/VnUserLink.vue';
import { useRole } from 'src/composables/useRole';
import VnTitle from 'src/components/common/VnTitle.vue';
Porque se modifica este componente reemplazando por VnTitle??
Porque es un summary, los summary gastan VnTitle. Si te fijas, gasta un router-link o un span con la clase link...
@ -10,3 +10,2 @@
import { useRole } from 'src/composables/useRole';
import VnTitle from 'src/components/common/VnTitle.vue';
Porque no se hace con el archivo siguiente, crear una funcion llamada getUrl??
No veo la necesidad, pero te lo cambio.
@ -67,3 +67,3 @@
const worker = computed(() => arrayData.store?.data);
const isHr = computed(() => hasAny(['hr']));
const isHr = computed(() => useRole().hasAny(['hr']));
duda: en router(index hemos rremplazado userole por useAcl, aqui proque no?
En este caso en concreto, no lo he tocado porque lo abordaré en otra tarea, ya que tengo que cambiar permisos y añadir otro acl. En el redmite está indicado.
@ -55,3 +55,2 @@
it(`should get redirected to dashboard since employee can't create tickets`, () => {
cy.visit('/#/ticket/create', { failOnStatusCode: false });
cy.url().should('contain', '/#/login?redirect=/ticket/create');
cy.visit('/#/account/ldap', { failOnStatusCode: false });
Hermos cambiado la URL pero no hemos cambiado el it del test...
@ -48,40 +48,62 @@ describe('useAcl', () => {
Test maravilloso 🤌