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

[TECH] Suppression du FT pour les écrans d'instruction certif v3 (PIX-12908) #10250

Merged
merged 1 commit into from
Oct 8, 2024

Conversation

mcampourcy
Copy link
Contributor

@mcampourcy mcampourcy commented Oct 4, 2024

🦄 Problème

Lors du développement des écrans d'instructions un FT a été mis en place que qu'ils n'apparaissent que sous condition. Le FT est désormais inutile

🤖 Proposition

Supprimer le FT FT_ENABLE_V3_INFO_SCREENS

💯 Pour tester

  • Se connecter à certif avec [email protected]
  • Aller sur mon pix et se connecter à une session de certification
  • Vérifier que les écrans d'instruction ne s'affichent pas
  • Se connecter à certif avec [email protected]
  • Aller sur sur mon pix et se connecter à une session de certification
  • Vérifier que les écrans d'instruction s'affichent

@mcampourcy mcampourcy self-assigned this Oct 4, 2024
@mcampourcy mcampourcy changed the title 🚛 mon-pix: Remove feature toggle for v3 instructions screens [TECH] Suppression du FT pour les écrans d'instruction certif v3 (PIX-12908) Oct 4, 2024
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@AndreiaPena
Copy link
Member

J'ai supprimé la variable de pix-api-review et de pix-api-review-pr10250

@alexandrecoin alexandrecoin force-pushed the pix-12908-remove-ft-for-v3-instructions-screens branch from 746ee41 to 25a464f Compare October 8, 2024 08:48
@alexandrecoin alexandrecoin merged commit 82e5ad8 into dev Oct 8, 2024
6 of 8 checks passed
@alexandrecoin alexandrecoin deleted the pix-12908-remove-ft-for-v3-instructions-screens branch October 8, 2024 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Func Review OK PO validated functionally the PR team-certif Tech Review OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants