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

Supprimer une phrase fausse dans le ReplyTransferMailer #2555

Merged
merged 3 commits into from
Jun 8, 2022

Conversation

francois-ferrandis
Copy link
Contributor

@francois-ferrandis francois-ferrandis commented Jun 8, 2022

Cette PR retire la phrase

Vous trouverez en pièce jointe l'email original.

des mails que l'on transfert aux agents en cas de réponse.

Cette phrase avait été ajoutée pendant le développement initial dans #2473, quand j'avais l'intention de fournir en pièce jointe une copie de la réponse, mais au final j'ai renoncé à la fonctionnalité mais j'ai oublié de virer cette phrase du template.

J'ai glissé dans la PR un commit qui désactive la concurrence dans sprockets, ce qui permet d'éviter un crash. Plus d'infos ici : rails/sprockets#581

I intended to attach the original email initially
but decided otherwise during development.
@francois-ferrandis francois-ferrandis self-assigned this Jun 8, 2022
@francois-ferrandis francois-ferrandis force-pushed the email-transfer-adjustments branch from 1a47fdd to 923b6e8 Compare June 8, 2022 07:29
@francois-ferrandis francois-ferrandis marked this pull request as ready for review June 8, 2022 07:30
@francois-ferrandis francois-ferrandis force-pushed the email-transfer-adjustments branch from 3606950 to 923b6e8 Compare June 8, 2022 07:52
@francois-ferrandis francois-ferrandis merged commit 6005b49 into production Jun 8, 2022
@francois-ferrandis francois-ferrandis deleted the email-transfer-adjustments branch June 8, 2022 08:09
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.

2 participants