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

Technique: Modification de l'email d'authentification des entreprises dans les fixtures #5457

Merged
merged 1 commit into from
Jan 24, 2025

Conversation

tonial
Copy link
Contributor

@tonial tonial commented Jan 24, 2025

🤔 Pourquoi ?

Pour pouvoir demander à rejoindre n'importe quelle entreprise sans risquer d'envoyer un email à n'importe qui

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?
  • Ajouter l'étiquette « Bug » ?

🏝️ Comment tester ?

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

@tonial tonial added the tech label Jan 24, 2025
@tonial tonial self-assigned this Jan 24, 2025
@xavfernandez
Copy link
Contributor

Est-ce qu'on est OK au niveau du GIP pour utiliser test+...@inclusion... ? Ou faudrait-il privilégier un emplois-test+...@inclusion... ?

@tonial
Copy link
Contributor Author

tonial commented Jan 24, 2025

bonne question, j'ai repris ce que @francoisfreitag a mis sur les utilisateurs

@tonial tonial force-pushed the alaurent/fix_fixtures branch from 62f9f41 to 796660a Compare January 24, 2025 10:07
@francoisfreitag
Copy link
Contributor

J’aurais bien ajouté la PK dans l’entreprise dans son email, histoire qu’on puisse plus facilement les distinguer. Genre [email protected].

We don't want to send invitation requests to real companies
@tonial tonial force-pushed the alaurent/fix_fixtures branch from 796660a to 777d2f3 Compare January 24, 2025 15:35
@tonial
Copy link
Contributor Author

tonial commented Jan 24, 2025

@francoisfreitag c'est fait

@tonial tonial added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC and removed 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC labels Jan 24, 2025
@tonial tonial added this pull request to the merge queue Jan 24, 2025
Merged via the queue into master with commit 932b815 Jan 24, 2025
12 of 14 checks passed
@tonial tonial deleted the alaurent/fix_fixtures branch January 24, 2025 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants