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

#6264 - renewToken #1824

Merged
jsegarra merged 36 commits from 6264-renewToken into dev 2023-12-20 09:22:29 +00:00
Conversation 0 Commits 36 Files Changed 11 +1 -2
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 61a323078b - Show all commits

3
back/methods/vn-user/renew-token.js
View File

@ -1,8 +1,7 @@
const UserError = require('vn-loopback/util/user-error');
const {models} = require('vn-loopback/server/server');
const DEFAULT_COURTESY_TIME = 60;
const handlePromiseLogout = (Self, {id}, courtesyTime = DEFAULT_COURTESY_TIME) => {
const handlePromiseLogout = (Self, {id}, courtesyTime) => {
jsegarra marked this conversation as resolved Outdated
jgallego commented 2023-12-13 06:32:15 +00:00
Outdated
Review
Copy Link

he visto el accessTokenConfig = 60 en las fixtures, no conviene quitarlo de aquí para no tener valores en el código?

he visto el accessTokenConfig = 60 en las fixtures, no conviene quitarlo de aquí para no tener valores en el código?
jsegarra commented 2023-12-13 09:42:31 +00:00
Outdated
Review
Copy Link

Se utiliza como valor por defecto del parámetro courtesyTime de una línea más abajo.

Esto se hace, por si, se diese el caso que el valor en BD está a nulo.

Se utiliza como valor por defecto del parámetro courtesyTime de una línea más abajo. Esto se hace, por si, se diese el caso que el valor en BD está a nulo.
new Promise(res => {
jsegarra marked this conversation as resolved Outdated
jgallego commented 2023-12-14 07:25:49 +00:00
Outdated
Review
Copy Link

tras la conversación asumimos que courtesy estará en la tabla definido

tras la conversación asumimos que courtesy estará en la tabla definido
setTimeout(() => {
res(Self.logout(id));