[8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription #58
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ption. Now you can know why an unsubscription was made.
Since they were not visible, I'm renaming them to under this module's namespace, but since it could have been used in production somewhere, I add a migration. The controller and view logic are simplified, images are added, README is improved, and some bugs are fixed by the way.
yajo
force-pushed
the
8.0-mass_mailing_custom_unsubscribe-reason
branch
from
3116e13
to
a079324
Compare
|
This one should be ready to review, please somebody label as such 😊 |
| @@ -8,10 +8,16 @@ Customizable unsubscription process on mass mailing emails | |||
| With this module you can set a custom unsubscribe link appended at the bottom | |||
| of mass mailing emails. | |||
|
|
|||
| It also displays a beautiful and simple unsubscription form when somebody | |||
| unsubscribes, to let you know why he left; and then displays to him a beautiful | |||
There was a problem hiding this comment.
Don't use he/him. Better to use an inpersonal style. Remove that parts from the sentence. The meaning is the same.
There was a problem hiding this comment.
It also displays a beautiful and simple unsubscription form when somebody unsubscribes, to let you know why; and then displays a beautiful...
|
All should be ready now. |
yajo
force-pushed
the
8.0-mass_mailing_custom_unsubscribe-reason
branch
from
2d1b70a
to
29d0ad9
Compare
yajo
force-pushed
the
8.0-mass_mailing_custom_unsubscribe-reason
branch
from
29d0ad9
to
8ba3676
Compare
| ====================== | ||
|
|
||
| * This module adds a security hash for mass mailing unsubscription URLs, which | ||
| makes URLs of mass mailing messages sent before its installation not work |
There was a problem hiding this comment.
s/makes URLs.../makes to not work anymore URLs... (and remove it from the end).
|
I got late here, but absolutely thank you @lasley! 🎉 |
yajo
added a commit
to Tecnativa/social
that referenced
this pull request
Nov 10, 2016
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
pedrobaeza
pushed a commit
to Tecnativa/social
that referenced
this pull request
Nov 25, 2016
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
pedrobaeza
pushed a commit
to Tecnativa/social
that referenced
this pull request
Jan 13, 2017
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
yajo
added a commit
to Tecnativa/social
that referenced
this pull request
Jan 25, 2017
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
yajo
added a commit
to Tecnativa/social
that referenced
this pull request
May 26, 2017
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
yajo
added a commit
to Tecnativa/social
that referenced
this pull request
May 30, 2017
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
yajo
added a commit
to Tecnativa/social
that referenced
this pull request
Jul 4, 2017
* [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
yajo
added a commit
to Tecnativa/social
that referenced
this pull request
Jul 10, 2017
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
yajo
added a commit
to Tecnativa/social
that referenced
this pull request
Jul 17, 2017
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
chienandalu
pushed a commit
to Tecnativa/social
that referenced
this pull request
May 25, 2018
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
chienandalu
pushed a commit
to Tecnativa/social
that referenced
this pull request
May 25, 2018
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
chienandalu
pushed a commit
to Tecnativa/social
that referenced
this pull request
Jun 11, 2018
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
ghost
pushed a commit
to camptocamp/social
that referenced
this pull request
Aug 21, 2018
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
ernestotejeda
pushed a commit
to Tecnativa/social
that referenced
this pull request
Jun 24, 2019
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
OCA-git-bot
pushed a commit
that referenced
this pull request
Jul 22, 2019
…ption (#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
jaag93
pushed a commit
to QubiQ/social
that referenced
this pull request
May 18, 2020
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
CarlosRoca13
pushed a commit
to Tecnativa/social
that referenced
this pull request
Aug 26, 2020
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
ernestotejeda
pushed a commit
to Tecnativa/social
that referenced
this pull request
Jul 23, 2021
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
ernestotejeda
pushed a commit
to Tecnativa/social
that referenced
this pull request
Jul 23, 2021
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
sergio-teruel
pushed a commit
to Tecnativa/social
that referenced
this pull request
Dec 7, 2022
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
ernestotejeda
pushed a commit
to Tecnativa/social
that referenced
this pull request
Dec 19, 2022
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
FernandoRomera
pushed a commit
to MallorcaSoft/social
that referenced
this pull request
Dec 20, 2022
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
FernandoRomera
pushed a commit
to MallorcaSoft/social
that referenced
this pull request
Dec 20, 2022
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
sergio-teruel
pushed a commit
to Tecnativa/social
that referenced
this pull request
Feb 1, 2023
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
carolinafernandez-tecnativa
pushed a commit
to Tecnativa/social
that referenced
this pull request
Feb 26, 2024
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
carolinafernandez-tecnativa
pushed a commit
to Tecnativa/social
that referenced
this pull request
Feb 26, 2024
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
pilarvargas-tecnativa
pushed a commit
to Tecnativa/social
that referenced
this pull request
Mar 1, 2024
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
chienandalu
pushed a commit
to Tecnativa/social
that referenced
this pull request
Jul 15, 2024
…ption (OCA#58) * [8.0][IMP][mass_mailing_custom_unsubscribe] Get reasons for unsubscription.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Now you can know why an unsubscription was made.
@Tecnativa.