? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
22 Jul 2019

Nothing major just tweaking the case and tense to be consistent on a few messages

Before

image

After

image

avatar brianteeman brianteeman - open - 22 Jul 2019
avatar brianteeman brianteeman - change - 22 Jul 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 22 Jul 2019
Category Libraries
avatar Quy
Quy - comment - 22 Jul 2019

I have tested this item successfully on bd0230c


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

avatar Quy Quy - test_item - 22 Jul 2019 - Tested successfully
avatar richard67
richard67 - comment - 22 Jul 2019

Screenshot for "After" seems not to be up to date.

avatar brianteeman
brianteeman - comment - 22 Jul 2019

possibly not

avatar richard67
richard67 - comment - 22 Jul 2019

Possibly yes: It shows 'Cleans expired cache entries', but in code is 'Clean expired cache entries'. Just 1 example.

avatar brianteeman
brianteeman - comment - 22 Jul 2019

the code is the truth

avatar richard67
richard67 - comment - 22 Jul 2019

I know, but correct testing instructions (including screenshots) are always good.

avatar richard67
richard67 - comment - 22 Jul 2019

With the PR applied, the output is still database:export Exports the database.

I know, this PR did not touch that code, but the screenshot showed so, and now the output is still inconsistent.

avatar brianteeman
brianteeman - comment - 22 Jul 2019

Thats because its in the vendor folder and I need to submit that upstream

avatar richard67
richard67 - comment - 22 Jul 2019

Unbenannt

avatar richard67
richard67 - comment - 22 Jul 2019

Ah, I see. Will you do that, submitt to upstream?

avatar brianteeman
brianteeman - comment - 22 Jul 2019

its already done

avatar richard67
richard67 - comment - 22 Jul 2019

I have tested this item successfully on bd0230c


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

avatar richard67 richard67 - test_item - 22 Jul 2019 - Tested successfully
avatar Quy Quy - change - 22 Jul 2019
Status Pending Ready to Commit
avatar Quy
Quy - comment - 22 Jul 2019

RTC


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

avatar brianteeman
brianteeman - comment - 23 Jul 2019

The changes required upstream have been merged

avatar wilsonge wilsonge - change - 23 Jul 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-07-23 17:32:25
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 23 Jul 2019
avatar wilsonge wilsonge - merge - 23 Jul 2019
avatar wilsonge
wilsonge - comment - 23 Jul 2019

Thanks!

avatar brianteeman
brianteeman - comment - 23 Jul 2019

Thanks

Add a Comment

Login with GitHub to post a comment