verdnatura/salix
verdnatura
/
salix
3
4
Fork 0
Code Issues Pull Requests 57 Releases Wiki Activity

Send message to comercial by RocketChat #1897

Merged
jsegarra merged 10 commits from 5499_claim_rocketMessage_comercial into dev 2024-01-11 07:00:13 +00:00
Conversation 0 Commits 10 Files Changed 6 +3 -3
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit b76e269e73 - Show all commits

6
modules/claim/back/methods/claim/updateClaim.js
View File

@ -110,7 +110,7 @@ module.exports = Self => {
}
};
async function notifyStateChange(ctx, id, claim, state) {
async function notifyStateChange(ctx, workerId, claim, newState) {
const models = Self.app.models;
jsegarra marked this conversation as resolved Outdated
jgallego commented 2024-01-10 09:21:40 +00:00
Outdated
Review
Copy Link

porque lo llamas id? no es el worker?

porque lo llamas id? no es el worker?
jsegarra commented 2024-01-10 09:25:56 +00:00
Outdated
Review
Copy Link

No puse worker porque en lo antiguo ,según el "code", usaba el id de salesPerson o de workerFk, y como ya no se distingue por code, puse un nombre genérico.

No puse worker porque en lo antiguo ,según el "code", usaba el id de salesPerson o de workerFk, y como ya no se distingue por code, puse un nombre genérico.
jgallego commented 2024-01-11 05:43:13 +00:00
Outdated
Review
Copy Link

hasta hoy si un archivo tiene id, y estamos en la ruta claim en este caso, se espera que eso sea claim id. En este caso es confuso, pon una referencia a worker

hasta hoy si un archivo tiene id, y estamos en la ruta claim en este caso, se espera que eso sea claim id. En este caso es confuso, pon una referencia a worker
👍 1
jsegarra commented 2024-01-11 06:29:46 +00:00
Outdated
Review
Copy Link

Corregido(b76e269e73)

Corregido(b76e269e737a63f6102a10ac831a4837424920c9)
const url = await models.Url.getUrl();
const $t = ctx.req.__; // $translate
@ -119,9 +119,9 @@ module.exports = Self => {
claimId: claim.id,
clientName: claim.client().name,
claimUrl: `${url}claim/${claim.id}/summary`,
newState: state
newState
});
await models.Chat.sendCheckingPresence(ctx, id, message);
await models.Chat.sendCheckingPresence(ctx, workerId, message);
}
async function notifyPickUp(ctx, workerId, claim) {