? ? Success

User tests: Successful: Unsuccessful:

avatar smehrbrodt
smehrbrodt
8 Jul 2015

This renames the "Purge" button on the Extensions->Update view from "Purge" to "Purge Cache" to make it clearer what the button does.

avatar smehrbrodt smehrbrodt - open - 8 Jul 2015
avatar smehrbrodt smehrbrodt - change - 8 Jul 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Jul 2015
Labels Added: ? ?
avatar brianteeman
brianteeman - comment - 8 Jul 2015

Personally I dont like the word purge in this specific use case and would prefer Clear Cache

avatar smehrbrodt
smehrbrodt - comment - 8 Jul 2015

Changed it to "Clear Cache". Sounds nicer indeed.

avatar brianteeman
brianteeman - comment - 8 Jul 2015

That means you will also need to update this string

COM_INSTALLER_MSG_DISCOVER_FAILEDTOPURGEEXTENSIONS="Failed to purge extensions"

avatar smehrbrodt
smehrbrodt - comment - 8 Jul 2015

That one is part of discover install, but surely I can change it also. What about these two:

JLIB_INSTALLER_PURGED_UPDATES="Purged updates"
JLIB_INSTALLER_FAILED_TO_PURGE_UPDATES="Failed to purge updates."

avatar brianteeman
brianteeman - comment - 8 Jul 2015

Personally I would look at ALL uses of the word purge and look to change
them to Clear AND add any extra wording required.

Purge is a DB specific word which while accurate is not as meaningful to
end users

On 8 July 2015 at 10:38, Samuel Mehrbrodt notifications@github.com wrote:

That one is part of discover install, but surely I can change it also.
What about these two:

JLIB_INSTALLER_PURGED_UPDATES="Purged updates"
JLIB_INSTALLER_FAILED_TO_PURGE_UPDATES="Failed to purge updates."


Reply to this email directly or view it on GitHub
#7376 (comment).

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar smehrbrodt
smehrbrodt - comment - 8 Jul 2015

Personally I would look at ALL uses of the word purge and look to change them to Clear AND add any extra wording required.

Ok, I've done that. Can you please check the changes?

avatar brianteeman
brianteeman - comment - 8 Jul 2015

Checked from my phone only but looks great to me - thank you

avatar brianteeman brianteeman - alter_testresult - 8 Jul 2015 - brianteeman: Not tested
avatar brianteeman brianteeman - alter_testresult - 8 Jul 2015 - brianteeman: Tested successfully
avatar brianteeman brianteeman - change - 8 Jul 2015
Category External Library Language & Strings
avatar Kubik-Rubik Kubik-Rubik - test_item - 8 Jul 2015 - Tested successfully
avatar Kubik-Rubik Kubik-Rubik - change - 8 Jul 2015
Labels Added: ?
avatar Kubik-Rubik
Kubik-Rubik - comment - 8 Jul 2015

Thank you @smehrbrodt! Merged.

avatar Kubik-Rubik Kubik-Rubik - change - 8 Jul 2015
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2015-07-08 11:41:55
Closed_By Kubik-Rubik
avatar Kubik-Rubik Kubik-Rubik - close - 8 Jul 2015
avatar zero-24 zero-24 - change - 14 Oct 2015
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment