-
Notifications
You must be signed in to change notification settings - Fork 36
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
(PC-30329)[API] feat: remove wip_enable_new_nav_ab_test ff and stop ff #12930
Conversation
315eb68
to
b0b2a45
Compare
) | ||
db.session.add(new_nav_pro) | ||
else: | ||
new_nav_pro = users_models.UserProNewNavState( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On pourrait supprimer la table UserProNewNavState ? Elle n'était utile que si le FF existait non ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
non puisqu'on veut garder l'ancienne NPP pour les utilisateurs invité. Et la modification ne concerne que les nouveaux utilisateurs. Il faut donc toujours faire la différence avec les actif non actifs. Le passage de tout les utilisateurs en NPP se fera surement avec une PR front. On fera différents tickets pour nettoyer ( routes, tables, fonctions)
@@ -0,0 +1,31 @@ | |||
"""remove_FF_WIP_ENABLE_NEW_NAV_AB_TEST |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suppression de FF faut plutôt faire une migration post
b738124
to
61e9f7d
Compare
@@ -0,0 +1,31 @@ | |||
"""remove_FF_WIP_ENABLE_NEW_NAV_AB_TEST |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"""remove_FF_WIP_ENABLE_NEW_NAV_AB_TEST | |
"""remove WIP_ENABLE_NEW_NAV_AB_TEST feature flag |
Comme pour
pass-culture-main/api/src/pcapi/alembic/versions/20240112T134955_3467ee234047_delete_stats_ff.py
Line 2 in 5499b0a
Removes WIP_HOME_STATS and WIP_HOME_STATS_V2 Feature Flag |
e1673c5
to
37d0035
Compare
no NPP activation by id parity and FF removal. The behavior for invitation remain
37d0035
to
440994f
Compare
Arret de l'AB test et suppression du FF associé.
Maintient de l'ancienne nav pour les utilisateurs invité par ceux qui l'ont.
But de la pull request
Ticket Jira (ou description si BSR) : https://passculture.atlassian.net/browse/PC-30329
Vérifications