#8031 - Enable notify positive when user update self data #750

Merged
jsegarra merged 10 commits from 8031_notify_dataSaved into dev 2024-09-30 11:35:19 +00:00
Member

image

![image](/attachments/6767e8b9-b9e5-4065-bf25-d022e7a8852d)
jsegarra added 1 commit 2024-09-23 18:49:25 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
a5c6d628ca
feat: enable notify positive when user update self data
jsegarra requested review from alexm 2024-09-23 18:49:27 +00:00
jsegarra added 1 commit 2024-09-24 20:17:57 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
bc37e372f5
Merge branch 'dev' into 8031_notify_dataSaved
jsegarra changed title from Enable notify positive when user update self data to #8031 - Enable notify positive when user update self data 2024-09-24 20:18:04 +00:00
alexm requested changes 2024-09-25 05:05:37 +00:00
Dismissed
@ -77,0 +79,4 @@
});
user.value.darkMode = value;
notify('globals.dataSaved', 'positive');
} catch (error) {
Member

Con un try catch para un console.log, el usuario no se dará cuenta si falla

Con un try catch para un console.log, el usuario no se dará cuenta si falla
jsegarra marked this conversation as resolved
@ -85,0 +92,4 @@
});
user.value.lang = value;
notify('globals.dataSaved', 'positive');
} catch (error) {
Member

Con un try catch para un console.log, el usuario no se dará cuenta si falla

Con un try catch para un console.log, el usuario no se dará cuenta si falla
jsegarra marked this conversation as resolved
@ -103,0 +115,4 @@
axios.post('UserConfigs/setUserConfig', { [param]: value });
localUserData();
notify('globals.dataSaved', 'positive');
} catch (error) {
Member

Con un try catch para un console.log, el usuario no se dará cuenta si falla

Con un try catch para un console.log, el usuario no se dará cuenta si falla
jsegarra marked this conversation as resolved
@ -103,2 +121,3 @@
}
const isEmployee = computed(() => useRole().isEmployee());
const onDataSaved = () => {
Member

No se llama nunca

No se llama nunca
jsegarra marked this conversation as resolved
jsegarra added 3 commits 2024-09-25 08:54:14 +00:00
jsegarra requested review from alexm 2024-09-25 08:58:49 +00:00
Author
Member

@alexm he añadido e2e

@alexm he añadido e2e
jsegarra added 1 commit 2024-09-25 12:08:42 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
73c45b8c4a
Merge branch 'dev' into 8031_notify_dataSaved
jsegarra added 1 commit 2024-09-26 12:17:43 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
a0b3e933a8
Merge branch 'dev' into 8031_notify_dataSaved
jsegarra added 1 commit 2024-09-27 18:02:26 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
12a452d774
Merge branch 'dev' into 8031_notify_dataSaved
jsegarra added the
CR / Tests passed
label 2024-09-27 18:02:33 +00:00
alexm approved these changes 2024-09-30 08:22:42 +00:00
jsegarra added 1 commit 2024-09-30 08:23:14 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
63d2b06b70
Merge branch 'dev' into 8031_notify_dataSaved
jsegarra added 1 commit 2024-09-30 11:34:41 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
7001a707b9
Merge branch 'dev' into 8031_notify_dataSaved
jsegarra scheduled this pull request to auto merge when all checks succeed 2024-09-30 11:34:45 +00:00
jsegarra merged commit 2d9dbbd23d into dev 2024-09-30 11:35:19 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 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-front#750
No description provided.