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 feature toggle FT_PIX1D_ENABLED #11624

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

aurelie-crouillebois
Copy link
Contributor

🥞 Problème

Un feature toggle avait été mis en place durant le développement de Pix Junior pour n'ouvrir l'accès aux routes destinées à Pix Junior que sur des plateformes internes.
Avec l'ouverture du service, ce feature toggle n'a plus d'intérêt.

🥓 Proposition

Le supprimer ainsi que tout le code s'y référant (security pre handler entre autres).

😋 Pour tester

Supprimer la variable d'environnement FT_PIX_1D_ENABLED de la review app et redémarrer.
Après redémarrage, l'application Pix Junior est toujours fonctionnelle : il est possible d'accéder à une école et de passer une mission.

@aurelie-crouillebois aurelie-crouillebois requested a review from a team as a code owner March 11, 2025 08:25
@aurelie-crouillebois aurelie-crouillebois added the feature-toggle Switch feature toggles to the new system label Mar 11, 2025
@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 :

@aurelie-crouillebois aurelie-crouillebois added 👀 Func Review Needed Need PO validation for this functionally and removed Func Review OK PO validated functionally the PR labels Mar 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👀 Func Review Needed Need PO validation for this functionally 👀 Tech Review Needed feature-toggle Switch feature toggles to the new system 🎒👧🏽👦🏾 PIX JUNIOR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants