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

[Formulaire signalement usager] Affichage des entreprises labellisées pour les territoires inactifs #353

Merged
merged 7 commits into from
Oct 6, 2023

Conversation

emilschn
Copy link
Collaborator

@emilschn emilschn commented Oct 3, 2023

Ticket

#352

Description

Contexte : Actuellement si un signalement est fait sur un territoire inactif, on lui envoie le protocole d'auto-traitement et rien d'autre.
Ce qu'on veut : On ajoute la possibilité d'accéder à la liste des entreprises labellisées dans le département de l'usager.

Changements apportés

  • Création d'une entité Entreprise Publique qui correspond aux entreprises qui sont affichées publiquement
  • Création d'une commande qui scrape un site pour récupérer la liste complète des entreprises dans un fichier csv
  • Création d'une commande qui parse un fichier csv et remplit la table des EntreprisePublique
  • Modification du parcours de formulaire usager quand le signalement est sur un territoire inactif
  • Envoi d'un e-mail spécifique pour ce cas

Pré-requis

Le fichier csv est déjà sur le cloud

Pour info : il n'est pas complet, il manque certains départements. Ca ne bloque pas les tests.

Tests

  • make console app="get-entreprise-publique"
  • Un fichier entreprises.csv est créé
  • make load-migrations
  • make console app="import-entreprise-publique"
  • La table EntreprisePublique est remplie
  • Faire un signalement sur un territoire inactif (ex : 44)
  • On propose de recevoir la liste des entreprises labellisées
  • On est redirigé vers la liste du département concerné
  • On peut trier les entreprises affichées
  • On reçoit un mail avec le template 23 et les bonnes données en paramètre

Tests de non-régression sur un territoire actif (ex : 13, 69)

  • Faire un signalement et déclarer un logement social
  • Faire un signalement et demander auto-traitement
  • Faire un signalement et demander traitement pro

templates/front/entreprises-labelisees.html.twig Outdated Show resolved Hide resolved
src/Service/Mailer/MailerProvider.php Outdated Show resolved Hide resolved
src/Service/Mailer/MailerProvider.php Outdated Show resolved Hide resolved
src/Service/Mailer/MailerProvider.php Outdated Show resolved Hide resolved
@hmeneuvrier
Copy link
Collaborator

Tests OK

@emilschn emilschn force-pushed the feature/352-entreprise-publique branch from 675cc15 to 37f79d1 Compare October 5, 2023 14:22
@hmeneuvrier
Copy link
Collaborator

tests OK

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.

C'est top ! Bien joué ! les tests sont ok aussi

Copy link
Collaborator

@sfinx13 sfinx13 left a comment

Choose a reason for hiding this comment

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

👍

@sfinx13 sfinx13 merged commit dabe0ea into develop Oct 6, 2023
3 checks passed
@emilschn emilschn deleted the feature/352-entreprise-publique branch October 6, 2023 14:13
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