No Code Attached Yet
avatar Quy
Quy
5 Oct 2023

mail-templates

Steps to reproduce the issue

Edit a mail template.
Insert a tag.
See browser console.

Additional comments

Not an issue in v4.4.

avatar Quy Quy - open - 5 Oct 2023
avatar joomla-cms-bot joomla-cms-bot - change - 5 Oct 2023
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 5 Oct 2023
avatar Quy Quy - change - 5 Oct 2023
Labels Added: Release Blocker
avatar Quy Quy - labeled - 5 Oct 2023
avatar richard67
richard67 - comment - 5 Oct 2023

Looks as if conditional chaining could also be helpful here ... ping @dgrammatiko .

avatar dgrammatiko
dgrammatiko - comment - 5 Oct 2023

I’ll have a look in a bit, unless @Fedik could patch it

avatar Fedik
Fedik - comment - 5 Oct 2023

It looks like the script missing editors dependency

avatar Fedik
Fedik - comment - 5 Oct 2023

Hm, no, I cannot reproduce it on latest 5.0-dev branch,
anything special need to do?

avatar dgrammatiko
dgrammatiko - comment - 5 Oct 2023

@Fedik you were right, #42058

avatar Fedik
Fedik - comment - 5 Oct 2023

okay

avatar richard67 richard67 - change - 5 Oct 2023
Status New Closed
Closed_Date 0000-00-00 00:00:00 2023-10-05 13:31:37
Closed_By richard67
avatar richard67 richard67 - close - 5 Oct 2023
avatar richard67
richard67 - comment - 5 Oct 2023

Closing as having a pull request. Please test #42058 . Thanks in advance.

avatar richard67 richard67 - change - 5 Oct 2023
Labels Removed: Release Blocker
avatar richard67 richard67 - unlabeled - 5 Oct 2023
avatar dgrammatiko
dgrammatiko - comment - 5 Oct 2023

@Fedik kinda irrelevant but could we skip the console.warn on the .min.js and only have it in the .js (debug)? I'm talking about this:
Screenshot 2023-10-05 at 16 33 23

avatar Fedik
Fedik - comment - 5 Oct 2023

Can skip, but I did not wanted to bother with it ;)

Add a Comment

Login with GitHub to post a comment