? Language Change ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
28 Sep 2021

when you have enabled notifications for workflows then the default message will be similar to


The status of an Article has been changed

Title: New feature: Workflows, changed by Brian Teeman, new state: Basic Stage.

Please log in to http://localhost/joomla-cms/administrator/index.php?option=com_messages&view=message&message_id=2 to read your message.

The last line implies that if you log in you will be able to read more about this message - and you might not even have the permissions to log in and read. However the online version of the message does not contain anything extra so there is no need to log in at all.

This PR changes the string to remove the information about logging in

The status of an Article has been changed

Title: New feature: Workflows, changed by Brian Teeman, new state: Basic Stage.

Pull request for #30361

avatar brianteeman brianteeman - open - 28 Sep 2021
avatar brianteeman brianteeman - change - 28 Sep 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Sep 2021
Category Administration Language & Strings
avatar Kostelano
Kostelano - comment - 28 Sep 2021

Yes, but this is a personal message template. And it contains tons of other available tags.

What is the point of using this template for workflows if, in fact, this template has become useless, because breaks another unrelated component...

avatar brianteeman
brianteeman - comment - 28 Sep 2021

99% sure that you are mistaken

avatar chmst
chmst - comment - 28 Sep 2021

Looks good for me. The message is only that a transition has been performed and is complete.

@Kostelano Sorry, I don't understand. Which template do you mean?

avatar Kostelano
Kostelano - comment - 28 Sep 2021

@chmst I'm talking about changing the email template. At the moment this template has other tags available that I can use in the body of the email. Considering that this template is also used for workflows (as we see in the original problem), it turns out to be a kind of mess - we'd better NOT use some tags, because they will be irrelevant in workflow emails.

Screenshot_1

avatar brianteeman
brianteeman - comment - 28 Sep 2021

The mail template takes its values from sql not the language string iirc. Either way even in that mail template it is a useless string for the same reasons mentioned above

avatar paternax
paternax - comment - 29 Sep 2021

The solution sounds good.
At the moment not possible to test, because the workflow in J.4.0.3. sends no notifications, even not to super user.

avatar brianteeman
brianteeman - comment - 29 Sep 2021

@paternax I was successfully getting emailed notifications BUT I suspect you hit another problem with notifications using com_messages in that it wont send a notification to you if you are the one that made the change and triggered the notification. Could you please create another super user and then see if they get the notifications

avatar paternax
paternax - comment - 29 Sep 2021

Yes you are right, that is exactly the problem. I tested this with two superusers and an explicit user in notifications. The same by using the usergroup superuser only the other superuser, who didn't made execute the transition gets an email.

avatar brianteeman
brianteeman - comment - 29 Sep 2021

@paternax which is logical but perhaps not expected

avatar brianteeman brianteeman - change - 20 Jan 2022
Title
[4.0] Log in to read messages
[4.1] Log in to read messages
avatar brianteeman brianteeman - edited - 20 Jan 2022
avatar brianteeman brianteeman - change - 20 Jan 2022
Labels Added: Language Change ? ?
avatar brianteeman
brianteeman - comment - 20 Jan 2022

rebased to 4.1

avatar Quy Quy - test_item - 8 Feb 2022 - Tested successfully
avatar Quy
Quy - comment - 8 Feb 2022

I have tested this item successfully on 56e181c


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35693.

avatar khu5h1 khu5h1 - test_item - 9 Feb 2022 - Tested successfully
avatar khu5h1
khu5h1 - comment - 9 Feb 2022

I have tested this item successfully on 56e181c


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35693.

avatar khu5h1
khu5h1 - comment - 9 Feb 2022

I have tested this item successfully on 56e181c


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35693.

avatar Quy Quy - change - 9 Feb 2022
Status Pending Ready to Commit
avatar Quy
Quy - comment - 9 Feb 2022

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/35693.

avatar bembelimen bembelimen - change - 20 Feb 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-02-20 18:23:43
Closed_By bembelimen
Labels Added: ?
Removed: ?
avatar bembelimen bembelimen - close - 20 Feb 2022
avatar bembelimen bembelimen - merge - 20 Feb 2022
avatar bembelimen
bembelimen - comment - 20 Feb 2022

Thx

avatar brianteeman
brianteeman - comment - 20 Feb 2022

thanks

Add a Comment

Login with GitHub to post a comment