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.

Sign up for GitHub

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

Jump to bottom

1.5.0 release #39

Merged
38 commits merged into from
Nov 12, 2020
Merged

1.5.0 release #39

38 commits merged into from
Nov 12, 2020

Conversation

ghost
Copy link

@ghost ghost commented Nov 12, 2020 •
edited by ghost
Loading

Plugin Version : 1.5.0

Version changelog

A list of changes regarding the next version release:

  1. Fix missing template variables - Missing template variables caused errors #34
  2. New feature to trigger opt-in if new customer joins with newsletter enabled - Trigger Automation If Customer Joins #35
  3. Use PrestashopLogger for logs created by module - Use PrestashopLogger for logging #38

Release checklist

  • Added release tag to this pull request
  • Updated README.md
  • Updated CHANGELOG.md
  • Updated USERGUIDE.md
  • Updated plugin version number
  • Updated screenshots in assets folder
  • Updated translations

After PR merge

  • Released new version in GitHub
  • Updated plugin on the Prestashop marketplace // N/A
  • Pinged code owners to inform marketing about new version

tomkabel added 30 commits November 2, 2020 18:27
@tomkabel
Missing template variables caused errors
2eeeb17
@tomkabel
on/off radio button, autoresponder select menu
8f33c6f
@tomkabel
Use same code for new autoresponder select menu
cc903da
@tomkabel
Add new config settings
9eaa717
@tomkabel
Reset credentials should clear optin autoresponder
ea7a9a0
@tomkabel
Save and display new config options properly
183b1f7
@tomkabel
Event to trigger opt-in if new account is made
f3c1955
@tomkabel
Merge pull request #34 from sendsmaily/dev/missing-template-variables
3d987ac 
Missing template variables caused errors
@tomkabel
Upgrade script was needed for config and hook
5e2a2b9
@tomkabel
Simplify configuration options
a8287e8
@tomkabel
Imporve wording for opt-in option
27e1cb7
@tomkabel
Add help block to opt-in option
f313285
@tomkabel
Improve wording for opt-in autoresponder
a55a62a
@tomkabel
Only save opt-in autoresponder ID to DB
63a67eb
@tomkabel
Option to choose no autoresponder for opt-in
d23c73b
@tomkabel
empty option I removed is needed for select menu
cb243b2
@tomkabel
Save only abandoned cart autoresponder ID to DB
51d45be
@tomkabel
Rephrase variable to be shorter
99f042c
@tomkabel
Upgrade script was not running
703f372
@tomkabel
Remove unused functions
9f4b53a
@tomkabel
Restore self-invoking anonymous function I removed
ca713ac 
I accidentally removed this in commit 63a67eb,
removing or restoring the function does not change its external behavior.
Autoresponders are fetched with or without it.
@tomkabel
Upgrade script required for abandoned cart values
be941d1
@tomkabel
Use more explanatory phrase for opt-in option
e4161f9
@tomkabel
Upgrade script didn't work
819163c
@tomkabel
Add translations
54722a6
@tomkabel
Missing &, extra i - update translations
473ec15
@tomkabel
Remove trailing dots
4cd6c4e
@tomkabel
Rephrase translation
1ec372e
@tomkabel
Dot was removed from help block
4b5b105
@tomkabel
Merge pull request #35 from sendsmaily/dev/trigger-automation-if-cust…
e84091d 
…omer-joins

Trigger Automation If Customer Joins
@ghost ghost added the release Released a new version label Nov 12, 2020
@ghost ghost requested a review from kaittodesk November 12, 2020 11:20
kaittodesk
kaittodesk requested changes Nov 12, 2020
View reviewed changes
@ghost ghost merged commit a13b817 into master Nov 12, 2020
@sinukaarel sinukaarel deleted the dev/1.5.0 branch April 4, 2024 05:17
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
release Released a new version
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kaittodesk @tomkabel