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

[FO - Page suivi usager] Revoir l'UI #2514

Merged
merged 15 commits into from
May 7, 2024

Conversation

numew
Copy link
Collaborator

@numew numew commented Apr 25, 2024

Ticket

#2149

Description

Mise en place de la nouvelle page de suivi usager (refonte v1)

Changements apportés

  • Refonte des pages usager "suivi signalement" et "suivi de la procédure"
  • Refonte de l'ajout de photo/document par l'usager via ces même pages
  • Fix affichage des messages d'erreur sur popup upload de document et du comportement en cas de fermeture rapide
  • Déplacement du fichier "Lettre information au bailleur"
  • Ajout d'une méthode getComplementAdresseOccupant sur Signalement pou factoriser l'affichage correspondant
  • Ajout d'un champ isTmpsur File (on n'affiche pas les doc ayant ce champ a true)
  • Suppression de la const SIGN_EXPORT inutilisé du SignalementVoter afin de le simplifier
  • Renommage des Front*Controller
  • Modification du contrôle d'accès au doc depuis le front pour les user logués

Pré-requis

make execute-migration name=Version20240426100957 direction=up
make npm-build

Tests

  • Créer un nouveau signalement et vérifier que la page de suivi usager affiche les informations correcte
  • Ajouter un suivi avec des document/photo et vérifier que tout fonctionne correctement
  • Visiter et tester la page de suivi de procédure d'un signalement
  • Visiter le suivi usager d'un signalement en version ancien formulaire et vérifier que la page s'affiche correctement

@numew numew marked this pull request as draft April 25, 2024 10:40
@numew numew marked this pull request as ready for review May 2, 2024 13:54
@numew numew force-pushed the feature/2149-ui-page-suivi-usager-v1 branch from 25fd555 to f4bc6ca Compare May 2, 2024 14:05
@numew numew mentioned this pull request May 3, 2024
2 tasks
@hmeneuvrier
Copy link
Collaborator

A part les détails que j'ai relevé, tests OK (hyper cool pour le suivi et l'abandon de procédure)

@emilschn
Copy link
Collaborator

emilschn commented May 6, 2024

image

Il faudrait un accent sur "déclarant" si possible :)

@emilschn
Copy link
Collaborator

emilschn commented May 6, 2024

Sinon, pareil qu'Hélène, mes tests sont ok ! C'est cool :)

@numew
Copy link
Collaborator Author

numew commented May 6, 2024

image

Il faudrait un accent sur "déclarant" si possible :)

Ok

Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK re-test et re-lecture

Copy link

sonarcloud bot commented May 7, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
5.5% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

Copy link
Collaborator

@hmeneuvrier hmeneuvrier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Relecture et re-tests OK pour moi

@hmeneuvrier hmeneuvrier merged commit cbbc96d into develop May 7, 2024
2 of 3 checks passed
@sfinx13 sfinx13 deleted the feature/2149-ui-page-suivi-usager-v1 branch May 7, 2024 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants