#7648 - Customer View Entries #2659
No reviewers
Labels
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix#2659
Loading…
Reference in New Issue
No description provided.
Delete Branch "7648_dev_customerEntries"
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: #7648 - Customer View Entriesto #7648 - Customer View Entries@ -0,0 +1,10 @@
-- Place your SQL code here
-- Auto-generated SQL script #202406281423
INSERT INTO salix.ACL (model,property,accessType,permission,principalType,principalId)
VALUES ('Entry','filter','READ','ALLOW','ROLE','$authenticated');
Creo que esto si eres un customer te dejaria ver todas la entries
solucion:
VALUES ('Entry','filter','READ','ALLOW','ROLE','supplier');
Crear rol supplier que hereda de account igual que customer.
a0185dca04
@ -36,1 +40,4 @@
if (isSupplier) {
const isEntryOwner = (await Self.findById(id)).supplierFk === ctx.req.accessToken.userId;
if (! isEntryOwner) throw new UserError('Access Denied');
!
@ -86,2 +107,3 @@
}]
};
delete filter.order;
quitar
@ -0,0 +1,28 @@
const UserError = require('vn-loopback/util/user-error');
eliminar fichero
@ -12,6 +12,7 @@ module.exports = Self => {
require('../methods/supplier/campaignMetricsEmail')(Self);
require('../methods/supplier/newSupplier')(Self);
require('../methods/supplier/getItemsPackaging')(Self);
require('../methods/supplier/isSupplier')(Self);
eliminar
@ -177,0 +180,4 @@
cu.code AS currencyCode,
t.shipped AS shipped,
t.landed AS landed,
t.warehouseInFk AS warehouseId,
se podria devolver el warehouseInFk tal cual sin el alias..lo apruebo porque no es un fallo, pero si lo ves mas conveniente lo cambias para la proxima
Lo cambio porque tampoco es correcto que t.shipped y t.landed lleven alias.
Además, queremos mostrar el almacén de entrada así que el alias de warehouseName no es correcto, debería ser warehouseInName,
Puede llevar a confusión, así que lo cambio
@ -3914,2 +3914,4 @@
VALUES(1, 1, 0, ' ', ' ', ' ', ' ', 0, '2001-01-01 00:00:00', 1, 0, ' ', ' ', 0, NULL, '', NULL, 0.000, NULL, 10, NULL, NULL, 'NCC', NULL);
UPDATE account.`user`
SET `role`=131
para que sirve esto?
hasta cuando se quedara en las fixtures?
para que el usuario customer de entorno local sea de tipo supplier, que hereda de customer.
Me parecía demasiado crear un usuario aposta, y como hereda roles, es transparente al desarrollador
Eliminado el cambio
beff905e02
@ -174,3 +178,3 @@
s.nickname AS supplierAlias,
co.code AS companyCode,
cu.code AS currencyCode
cu.code AS currencyCode,
por convencion va sin AS
cu.code currencyCode,
esta linea y las demás tb
946d6f553a