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

[BO - Utilisateur] UI / UX pour utilisateurs multi-territoire #3554

Merged
merged 6 commits into from
Jan 23, 2025

Conversation

numew
Copy link
Collaborator

@numew numew commented Jan 16, 2025

Ticket

#3308

Description

  • Ajout du feature flipping FEATURE_MULTI_TERRITORIES
  • Ajout d'un agent à un partenaire : Nouvelle version (modale en deux étapes). Multi territoire impossible sur user existant si SA/RT ou ayant les permissions d'affectation
  • Edition d'un agent : Edition bloqué sur un agent multi territoire
  • Suppression d'un agent du territoire : Ajout d'un email spécifique lors de la suppression d'un agent d'un partenaire dans le cas ou il reste actif sur d'autre territoire

Pré-requis

FEATURE_MULTI_TERRITORIES=1
make load-fixtures

Tests

  • Tester l'ajout d'un agent à un partenaire
  • Tester l'ajout d'un agent existant sur un autre territoire à un partenaire
  • Vérifier l'ajout d'agent à un partenaire sur la version FEATURE_MULTI_TERRITORIES=0
  • Tester l'édition d'un agent sur un partenaire
  • Tester la suppression d'un agent multi-territoire d'un partenaire

@numew numew force-pushed the feature/3308-ui-multi-ter branch from 1d7b17e to ddf263d Compare January 16, 2025 10:02
@numew numew changed the title [BO - Utilisateur] UI / UX pour utilisateurs multi-territoire [WIP] [BO - Utilisateur] UI / UX pour utilisateurs multi-territoire Jan 17, 2025
@numew numew force-pushed the feature/3308-ui-multi-ter branch 2 times, most recently from e912ef4 to 1b5515e Compare January 17, 2025 12:47
@numew numew changed the title [WIP] [BO - Utilisateur] UI / UX pour utilisateurs multi-territoire [BO - Utilisateur] UI / UX pour utilisateurs multi-territoire Jan 17, 2025
@numew numew marked this pull request as ready for review January 17, 2025 14:59
@numew numew requested review from emilschn, hmeneuvrier and sfinx13 and removed request for emilschn January 17, 2025 14:59
}
})
}else{
const content = '<div class="fr-alert fr-alert--error" role="alert"><p class="fr-alert__title">Erreur</p><p>Une erreur est survenue veuillez fermer cette fenêtre.</p></div>'
Copy link
Collaborator

Choose a reason for hiding this comment

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

On pourrait faire plus précis ? Genre fermer la fenêtre, rafraichir la page, et réessayer ? J'imagine que ça peut se produire en cas de perte de session ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ok (j'ai repris le message qu'on utilise a chaque fois)


return $this->json(['redirect' => true, 'url' => $url]);
}
$formMultiMail->get('email')->addError(new FormError('Agent introuvalbe avec cette adresse e-mail.'));
Copy link
Collaborator

Choose a reason for hiding this comment

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

introuvable

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ok, merci

</table>
<p>Si le bouton ne fonctionne pas <a href="{{ link|raw }}" target="_blank" rel="noopener">cliquez ici</a></p>
<p>
Pour plus d'information, rapprochez vous de votre référent territorial.
Copy link
Collaborator

Choose a reason for hiding this comment

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

rapprochez-vous

Copy link
Collaborator

Choose a reason for hiding this comment

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

on peut mettre "responsable de territoire", pour garder le même terme partout ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

J'ai pas touché celui-la vu le terme utilisé dans les autre e-mail

Copy link
Collaborator

Choose a reason for hiding this comment

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

on peut mettre "responsable de territoire" dans tous les mails ? :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

bon d'accord

Copy link
Collaborator

Choose a reason for hiding this comment

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

Hum, étant donné les choix précédents, j'aurais vérifié avec Mathilde avant. Si elle a proposé ce terme-là dans les e-mails, ça doit pas être pour rien, à mon avis !

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.

Quelques commentaires de typo. Sinon relecture et tests ok de ce que j'ai vu.

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.

Tests OK

@@ -15,9 +15,8 @@
<div class="fr-select-group">
Vous êtes sur le point de supprimer le compte utilisateur de <span class="fr-modal-user-delete_username"></span> (<span class="fr-modal-user-delete_useremail"></span>). Une fois le compte supprimé :
<ul class="fr-list">
<li><span class="fr-modal-user-delete_username"></span> ne pourra plus se connecter à la plateforme {{ platform.name }} pour traiter des signalements.</li>
<li><span class="fr-modal-user-delete_username"></span> ne pourra plus traiter des signalements sur le territoire.</li>
Copy link
Collaborator

Choose a reason for hiding this comment

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

s'il n'a qu'un partenaire, ou si c'est le dernier partenaire, il ne pourra plus se connecter non plus, si ?

</table>
<p>Si le bouton ne fonctionne pas <a href="{{ link|raw }}" target="_blank" rel="noopener">cliquez ici</a></p>
<p>
Pour plus d'information, rapprochez vous de votre référent territorial.
Copy link
Collaborator

Choose a reason for hiding this comment

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

on peut mettre "responsable de territoire" dans tous les mails ? :)

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.

ok pour moi

Copy link

Quality Gate Failed Quality Gate failed

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

See analysis details on SonarQube Cloud

@numew numew merged commit e136293 into develop Jan 23, 2025
3 of 4 checks passed
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.

3 participants