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
[DPMMA-2341] Fixed missing preference center #12829
Conversation
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.
while the code has the correct outcome, it differs from the approach in M4, see https://github.com/mautic/mautic/blob/4.4.10/app/bundles/EmailBundle/Views/Email/form.html.php#L224...L227
in M4 one of the 2 permissions was enough, in M5 both are needed
Co-authored-by: Mattias Michaux <mattias.michaux@gmail.com>
Codecov Report
@@ Coverage Diff @@
## 5.x #12829 +/- ##
=========================================
Coverage 58.67% 58.68%
Complexity 33631 33631
=========================================
Files 2181 2181
Lines 101824 101824
=========================================
+ Hits 59750 59751 +1
+ Misses 42074 42073 -1 |
@mollux thanks for catching it! I commited the change. |
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.
I noticed same during the build #12860
This fixed it 👍
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.
Works well.
Thanks. I faced the same problem some time ago.
This pull request has been mentioned on Mautic Forums. There might be relevant details there: https://forum.mautic.org/t/announcing-the-mautic-5-release-candidate/29967/1 |
Description:
Steps to test this PR: