verdnatura/hedera-web
verdnatura
/
hedera-web
0
0
Fork 1
Code Issues Pull Requests 1 Releases Wiki Activity

Account config and change password form #73

Merged
jsegarra merged 10 commits from wbuezas/hedera-web-mindshore:feature/AccountConfig into 4922-vueMigration 2024-07-26 20:24:42 +00:00
Conversation 0 Commits 10 Files Changed 11 +535 -41
wbuezas commented 2024-07-25 14:43:38 +00:00
Contributor
Copy Link
No description provided.
wbuezas added 3 commits 2024-07-25 14:43:38 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
401487dfd3 Resolve conflicts
wbuezas requested review from jsegarra 2024-07-25 14:43:50 +00:00
jsegarra requested changes 2024-07-26 05:37:43 +00:00
Dismissed
src/components/ui/ChangePasswordForm.vue
@ -0,0 +1,202 @@
<script setup>
jsegarra commented 2024-07-26 05:01:26 +00:00
Member
Copy Link

Revisamos el component porque faltan detalles como:

  1. Que pasa cuando algún campo está vacío? Deberia estar la comprobacion, igual que se hace en forms/account/conf/index.js
  2. Que pasa si tenemos verificationToken? Deberia estar la lógica
  3. Que pasa cuando hemos cambiado la contraseña? Deberia aparecer un mensaje
Revisamos el component porque faltan detalles como: 1. Que pasa cuando algún campo está vacío? Deberia estar la comprobacion, igual que se hace en forms/account/conf/index.js 2. Que pasa si tenemos verificationToken? Deberia estar la lógica 3. Que pasa cuando hemos cambiado la contraseña? Deberia aparecer un mensaje
wbuezas commented 2024-07-26 12:11:00 +00:00
Author
Contributor
Copy Link
  1. Se agregaron las validaciones faltantes.
    Commit: 06cd9b01d3

Ahora en lugar de desloguear cuando se cambia la contraseña hace un login automatico, como sucede en hedera.
Commit: 93cc0d4286

Añadí el verificationToken como prop para desacoplar al formulario de cambiar contraseña de esa logica y dejar que la vista que contenga el form maneje esa parte.
Commit: 4256f45373

  1. Se agregó toda la lógica relacionada al auth token faltante.
    Commit: ef36566442

  2. Cuando el submit que lanza VnForm.vue es exitoso ya muestra un toast de Data saved

1. Se agregaron las validaciones faltantes. `Commit:` https://gitea.verdnatura.es/verdnatura/hedera-web/commit/06cd9b01d363aa0bc1bb1a4f7de0f65479d7c7ed Ahora en lugar de desloguear cuando se cambia la contraseña hace un login automatico, como sucede en hedera. `Commit:` https://gitea.verdnatura.es/verdnatura/hedera-web/commit/93cc0d428640692f82d3d1d5cd5f50d18a33e884 Añadí el `verificationToken` como `prop` para desacoplar al formulario de cambiar contraseña de esa logica y dejar que la vista que contenga el form maneje esa parte. `Commit:` https://gitea.verdnatura.es/verdnatura/hedera-web/commit/4256f45373d92961c4da6dd3f7714ba132110f5d 2. Se agregó toda la lógica relacionada al auth token faltante. `Commit:` https://gitea.verdnatura.es/verdnatura/hedera-web/commit/ef365664423e16c08813ad353afa2ec4eb819fff 3. Cuando el submit que lanza `VnForm.vue` es exitoso ya muestra un toast de `Data saved`
src/pages/Account/AccountConfig.vue
@ -0,0 +108,4 @@
</QPage>
</template>
<style lang="scss" scoped></style>
jsegarra commented 2024-07-26 05:37:38 +00:00
Member
Copy Link

Si no hay reglas CSS, la quitamos

Si no hay reglas CSS, la quitamos
wbuezas commented 2024-07-26 11:41:50 +00:00
Author
Contributor
Copy Link

Tag de style quitada.

Commit: 7f831ae3a5

Tag de style quitada. Commit: https://gitea.verdnatura.es/verdnatura/hedera-web/commit/7f831ae3a54a05349212b1b4a4a2806374173dc5
wbuezas added 1 commit 2024-07-26 11:41:27 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
7f831ae3a5 Remove unused style tag
wbuezas added 1 commit 2024-07-26 11:54:20 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
06cd9b01d3 Change password form fields validation
wbuezas added 1 commit 2024-07-26 12:09:29 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
ef36566442 Create change password with and without token and add related features
wbuezas added 1 commit 2024-07-26 12:25:54 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
93cc0d4286 Add login when password changed
wbuezas added 1 commit 2024-07-26 13:36:57 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
4256f45373 Add verificationToken as a prop to let the view handle it
jsegarra added 1 commit 2024-07-26 20:18:22 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration There was a failure building this commit Details
6bad41db20 feat: add password visibility
jsegarra approved these changes 2024-07-26 20:21:35 +00:00
jsegarra added 1 commit 2024-07-26 20:23:25 +00:00
gitea/hedera-web/pipeline/pr-4922-vueMigration This commit looks good Details
c20f48b2bf Merge branch '4922-vueMigration' into feature/AccountConfig
jsegarra merged commit 24687e57e6 into 4922-vueMigration 2024-07-26 20:24:42 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
jsegarra
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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/hedera-web#73
No description provided.
Delete Branch "wbuezas/hedera-web-mindshore:feature/AccountConfig"

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?