? ? Pending

User tests: Successful: Unsuccessful:

avatar richard67
richard67
3 Jun 2021

Pull Request for Issue # .

Summary of Changes

Adapt a language string for the pre-update check to changes from merged PR #34366 .

Testing Instructions

Code review, or make that a database error is shown by the database check e.g. by adding a new, empty update SQL script "4.0.0-2021-06-03.sql", and then checking the pre-update check.

Actual result BEFORE applying this Pull Request

2021-06-03-pre-update-check

Expected result AFTER applying this Pull Request

The right way to the database check is shown.

Documentation Changes Required

None.

avatar richard67 richard67 - open - 3 Jun 2021
avatar richard67 richard67 - change - 3 Jun 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 3 Jun 2021
Category Administration Language & Strings
avatar richard67
richard67 - comment - 3 Jun 2021

@brianteeman Please test.

753553f 3 Jun 2021 avatar richard67 Oops
avatar richard67 richard67 - change - 3 Jun 2021
Labels Added: ? ?
avatar brianteeman
brianteeman - comment - 3 Jun 2021

Well remembered - thanks for fixing.

I "think" the other string is used in the alternate menu? If so then it doesnt need changing


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

avatar brianteeman brianteeman - test_item - 3 Jun 2021 - Tested successfully
avatar brianteeman
brianteeman - comment - 3 Jun 2021

I have tested this item successfully on 753553f


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

avatar Quy Quy - change - 3 Jun 2021
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-06-03 16:46:39
Closed_By Quy
avatar Quy Quy - close - 3 Jun 2021
avatar Quy Quy - merge - 3 Jun 2021
avatar Quy
Quy - comment - 3 Jun 2021

Thanks!

avatar richard67
richard67 - comment - 3 Jun 2021

Thanks!

Add a Comment

Login with GitHub to post a comment