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

Désactive l'ab-testing sur l'email d'envoi du sondage #4080

Merged
merged 1 commit into from
Nov 29, 2023

Conversation

Shamzic
Copy link
Contributor

@Shamzic Shamzic commented Nov 24, 2023

Pré-requis

#4066 (refacto des services email/sms)

Description

L'AB testing sur l'envoi des emails de sondage est désactivé.

Les reliquats du template supprimé (benefit-action) sont conservés pour le moment car les liens contenus dans les emails envoyés avant cette modification doivent pouvoir continuer à fonctionner.

@Shamzic Shamzic self-assigned this Nov 24, 2023
@guillett guillett added this to the BC milestone Nov 24, 2023
Copy link
Contributor

@baptou12 baptou12 left a comment

Choose a reason for hiding this comment

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

Je pense qu'on devrait aussi enlever le "singleEmailType" du coup :

// Single emails types parsers
const singleEmailTypes = [
EmailType.SimulationResults,
EmailType.BenefitAction,
EmailType.SimulationUsefulness,
]
singleEmailTypes.forEach((emailType) => {
const parser = send_types.add_parser(emailType)
parser.add_argument("--id", {
help: "Followup Id",
})
})

Je comprend bien qu'on ne puisse pas enlever le render et la gestion du tracking en ce moment car ça invaliderait beaucoup de liens / visualisation d'email.

Ça serai pas mal d'écrire le plan et de réfléchir à quand le dérouler à l'avenir pour cleaner a 100% ? (p.e. avec migration?) T'en pense quoi ?

@Shamzic
Copy link
Contributor Author

Shamzic commented Nov 27, 2023

Je pense qu'on devrait aussi enlever le "singleEmailType" du coup :

On peut enlever l"option EmailType.SimulationUsefulness, mais je trouve ça toujours pratique de pouvoir choisir d'envoyer manuellement la simulation et / ou le sondage indépendamment.

Ça serai pas mal d'écrire le plan et de réfléchir à quand le dérouler à l'avenir pour cleaner a 100% ? (p.e. avec migration?) T'en pense quoi ?

Oui, il faut déjà déterminer une date adéquate pou faire ce clean complet. Il y aurait effectivement une migration à faire. J'ajoute un ticket à prioriser pour ce sujet : https://trello.com/c/mCOPKeXc/1531-supprimer-les-anciens-types-de-sondages-clean-reliquats-ab-testing

@baptou12
Copy link
Contributor

On peut enlever l"option EmailType.SimulationUsefulness, mais je trouve ça toujours pratique de pouvoir choisir d'envoyer manuellement la simulation et / ou le sondage indépendamment.

Je pensais plus à enlevé BenefitAction de singleEmailType vu qu'il n'est plus présent dans processSingleEmail

@Shamzic
Copy link
Contributor Author

Shamzic commented Nov 27, 2023

On peut enlever l"option EmailType.SimulationUsefulness, mais je trouve ça toujours pratique de pouvoir choisir d'envoyer manuellement la simulation et / ou le sondage indépendamment.

Je pensais plus à enlevé BenefitAction de singleEmailType vu qu'il n'est plus présent dans processSingleEmail

Oui pardon c'est l'inverse !

@Shamzic Shamzic force-pushed the desactiver-email-benefit-action branch from 94e3dc8 to 1e24b97 Compare November 27, 2023 11:04
@Shamzic Shamzic requested a review from baptou12 November 27, 2023 12:17
@Shamzic Shamzic force-pushed the desactiver-email-benefit-action branch from 1e24b97 to 55d1fd8 Compare November 27, 2023 12:19
Base automatically changed from refactor-email-sms to master November 28, 2023 15:42
@Shamzic Shamzic force-pushed the desactiver-email-benefit-action branch from 55d1fd8 to 5a12571 Compare November 28, 2023 16:25
@Shamzic Shamzic merged commit 40fc52b into master Nov 29, 2023
24 checks passed
@Shamzic Shamzic deleted the desactiver-email-benefit-action branch November 29, 2023 09:05
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