refs #6434 feat: improve signIn method #1867

Merged
jsegarra merged 11 commits from 6434-signInLog_improve_test into test 2023-11-30 11:20:58 +00:00
Member
No description provided.
jsegarra added 1 commit 2023-11-28 12:23:17 +00:00
gitea/salix/pipeline/head This commit looks good Details
63533459b0
refs #6434 feat: improve signIn method
jsegarra requested review from alexm 2023-11-28 12:23:59 +00:00
alexm reviewed 2023-11-29 06:36:54 +00:00
@ -0,0 +1,15 @@
Member

Normalment no habriem ficat 234802... pero bueno

Normalment no habriem ficat 234802... pero bueno
jsegarra marked this conversation as resolved
alexm approved these changes 2023-11-29 06:37:14 +00:00
Dismissed
jgallego requested changes 2023-11-29 08:30:43 +00:00
@ -128,0 +127,4 @@
/**
* Sign-in validate. *
* @param {Integer} user The user
Owner

{string}

{string}
@ -131,0 +139,4 @@
token: token.id,
userFk: userToken.id,
ip: ctx.req.ip,
owner: isOwner
Owner

userName : user

userName : user
jsegarra marked this conversation as resolved
@ -131,0 +142,4 @@
owner: isOwner
});
if (!isOwner) {
console.error('ERROR!!! - SignIn with other user', userToken, user);
Owner

quitar

quitar
@ -0,0 +8,4 @@
`userFk` int(10) unsigned DEFAULT NULL,
`creationDate` timestamp NULL DEFAULT current_timestamp(),
`ip` varchar(100) CHARACTER SET utf8mb3 COLLATE utf8mb3_general_ci NOT NULL,
`owner` tinyint(1) DEFAULT 1,
Owner

añadir userName varchar(30) NOT NULL

añadir userName varchar(30) NOT NULL
jsegarra marked this conversation as resolved
jsegarra added 1 commit 2023-11-29 19:32:11 +00:00
gitea/salix/pipeline/head This commit looks good Details
48b82b02ca
refs #6434 change param type for signInValidate
jsegarra dismissed alexm’s review 2023-11-29 19:32:11 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

jsegarra added 1 commit 2023-11-29 19:33:06 +00:00
gitea/salix/pipeline/head Build queued... Details
ed6c0924d7
refs #6434 perf: remove console.error
jsegarra added 1 commit 2023-11-29 19:34:01 +00:00
gitea/salix/pipeline/head This commit looks good Details
9176cdb4cb
refs #6434 perf: new field in SignInLog table
jsegarra added 1 commit 2023-11-29 19:35:40 +00:00
jsegarra added 1 commit 2023-11-29 19:36:33 +00:00
gitea/salix/pipeline/head This commit looks good Details
1576ab992d
refs #6434 perf: new field in SignInLog table
jsegarra added 1 commit 2023-11-30 07:11:45 +00:00
gitea/salix/pipeline/head There was a failure building this commit Details
138d11b7b0
refs #6434 perf new version folder for sql file
jsegarra added 1 commit 2023-11-30 07:13:40 +00:00
gitea/salix/pipeline/head Build queued... Details
6b354a20ad
refs #6434 perf: add sql table description
jsegarra requested review from jgallego 2023-11-30 07:13:53 +00:00
jsegarra added 1 commit 2023-11-30 07:15:07 +00:00
gitea/salix/pipeline/head There was a failure building this commit Details
58855a7cdd
refs #6434 perf: remove bad characters method description
jsegarra added 1 commit 2023-11-30 07:23:26 +00:00
gitea/salix/pipeline/head This commit looks good Details
80f8037f58
refs #6434 perf remove version folder for sql file
jsegarra added 1 commit 2023-11-30 07:34:21 +00:00
gitea/salix/pipeline/head This commit looks good Details
7611acb8f2
refs #6434 perf rename version folder for sql file
alexm approved these changes 2023-11-30 07:50:22 +00:00
jgallego approved these changes 2023-11-30 08:00:42 +00:00
jsegarra merged commit b241efc961 into test 2023-11-30 11:20:58 +00:00
jsegarra deleted branch 6434-signInLog_improve_test 2023-11-30 11:20:58 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/salix#1867
No description provided.