Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Story #12069 feat(archive-search): add archive unit edit mode #1655

Merged

Conversation

Regzox
Copy link
Contributor

@Regzox Regzox commented Feb 23, 2024

Description

Ajout formulaire de mise à jour dynamique pour la mise à jour d'unité d'archive.

@Regzox Regzox marked this pull request as draft February 23, 2024 16:58
@Regzox Regzox force-pushed the story_12069__update_archive_unit_descriptive_metadata branch from 07b1fa9 to d1a7c71 Compare February 23, 2024 17:03
@vitam-devops
Copy link
Collaborator

vitam-devops commented Feb 23, 2024

Logo
Checkmarx One – Scan Summary & Details469cf3a4-bb04-482d-95a3-4214e05bf712

New Issues

Severity Issue Source File / Package Checkmarx Insight
LOW Use_Of_Hardcoded_Password /ui/ui-frontend-common/src/app/modules/object-editor/services/template.service.spec.ts: 151 Attack Vector
LOW Use_Of_Hardcoded_Password /ui/ui-frontend-common/src/app/modules/object-editor/services/template.service.spec.ts: 143 Attack Vector
LOW Use_Of_Hardcoded_Password /ui/ui-frontend-common/src/app/modules/object-editor/services/template.service.spec.ts: 131 Attack Vector
LOW Use_Of_Hardcoded_Password /ui/ui-frontend-common/src/app/modules/object-editor/services/template.service.spec.ts: 122 Attack Vector
LOW Use_Of_Hardcoded_Password /ui/ui-frontend-common/src/app/modules/object-editor/services/template.service.spec.ts: 68 Attack Vector
LOW Use_Of_Hardcoded_Password /ui/ui-frontend-common/src/app/modules/object-editor/services/template.service.spec.ts: 58 Attack Vector
LOW Use_Of_Hardcoded_Password /ui/ui-frontend-common/src/app/modules/object-editor/services/template.service.spec.ts: 51 Attack Vector
LOW Use_Of_Hardcoded_Password /ui/ui-frontend-common/src/app/modules/object-editor/services/template.service.spec.ts: 41 Attack Vector

@Regzox Regzox force-pushed the story_12069__update_archive_unit_descriptive_metadata branch from e74fd7e to 4589bc6 Compare March 1, 2024 13:30
@Regzox Regzox marked this pull request as ready for review March 1, 2024 15:43
@Regzox Regzox requested review from marob and laedanrex March 1, 2024 15:44
@GiooDev GiooDev added this to the IT 132 milestone Mar 1, 2024
@GiooDev GiooDev changed the title Story 12069 update archive unit descriptive metadata Story #12069: Update archive unit descriptive metadata Mar 1, 2024
@marob
Copy link
Contributor

marob commented Mar 4, 2024

Le texte devrait être "Modifier les métadonnées descriptives"

image

@marob
Copy link
Contributor

marob commented Mar 4, 2024

Est-ce normal d'avoir un tooltip avec du JSON au rollover des titres ?

image

@marob
Copy link
Contributor

marob commented Mar 4, 2024

Problème d'espaces sur le bandeau sticky

image

@marob
Copy link
Contributor

marob commented Mar 4, 2024

Il manque le "Services producteurs liés à l’unité archivistique" qui est présent en mode consultation.
Aussi, ces champs devraient être non modifiables et grisés (ce n'est pas le cas).
Il y a également une inconsistance sur le nommage des champs entre l'US/Figma et l'application Vitam UI.

image

@marob
Copy link
Contributor

marob commented Mar 4, 2024

Le "Niveau de description", présent en consultation, est manquant.

image

@marob
Copy link
Contributor

marob commented Mar 4, 2024

  • je ne trouve pas de bloc "PersistentIdentifier"
  • le bouton "+" devrait être un "..." vertical avec choix et ne correspond pas au Figma (couleur, design, ...)
  • le gris utilisé pour les composant vides devrait être différent du gris des composants non vides et repliés

image

@marob
Copy link
Contributor

marob commented Mar 4, 2024

Lorsqu'aucune modification n'a été effectuée, le bouton enregistrer, bien que non cliquable, semble l'être. Son style n'est pas cohérent. Par ailleurs, le bouton annuler ne devrait pas ouvrir de popin si aucune modification n'a été effectuée.
image

Lorsqu'une modification a été effectuée, le bouton enregistrer est bien cliquable, mais son style n'est pas cohérent.
image

Lorsque les modifications sont enregistrées, l’onglet Description ne repasse pas en mode affichage.

@marob
Copy link
Contributor

marob commented Mar 4, 2024

La popin de confirmation d'une annulation de modification n'est pas déclenchée dans les cas suivants :

  • clic sur un autre onglet
  • clic sur la croix permettant de fermer le panneau latéral
  • clic n'importe où ailleurs que dans le panneau latéral

À noter qu'une fois la popin fermée (confirmation d'annulation ou enregistrement), l'action initiale doit être déclenchée (aller sur l'onglet/fermer le panneau/charger une autre Unit/...)

@Regzox Regzox force-pushed the story_12069__update_archive_unit_descriptive_metadata branch from 5125cdc to 6d4576c Compare March 4, 2024 13:19
@Regzox Regzox force-pushed the story_12069__update_archive_unit_descriptive_metadata branch from 6d4576c to 04d8550 Compare March 5, 2024 13:45
@Regzox Regzox force-pushed the story_12069__update_archive_unit_descriptive_metadata branch 4 times, most recently from db14cc5 to 327ed8f Compare March 6, 2024 17:18
@Regzox Regzox changed the title Story #12069: Update archive unit descriptive metadata Story #12069 feat(archive-search): add archive unit edit mode Mar 8, 2024
@Regzox Regzox added enhancement New feature or request javascript Pull requests that update Javascript code java Pull requests that update Java code labels Mar 8, 2024
@Regzox Regzox added the VAS VAS contribution label Mar 8, 2024
@Regzox Regzox force-pushed the story_12069__update_archive_unit_descriptive_metadata branch 4 times, most recently from 1a36ea5 to 407d023 Compare March 8, 2024 16:34
@Regzox Regzox force-pushed the story_12069__update_archive_unit_descriptive_metadata branch 3 times, most recently from 48033a5 to cffd065 Compare March 11, 2024 17:43
@Regzox Regzox force-pushed the story_12069__update_archive_unit_descriptive_metadata branch from cffd065 to f211a19 Compare March 11, 2024 17:55
@marob marob merged commit f211a19 into develop Mar 12, 2024
2 checks passed
@marob marob deleted the story_12069__update_archive_unit_descriptive_metadata branch March 12, 2024 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request java Pull requests that update Java code javascript Pull requests that update Javascript code VAS VAS contribution
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants