Skip to content

Conversation

@richard67
Copy link
Member

@richard67 richard67 commented Apr 2, 2023

Pull Request for Issue # .

Summary of Changes

This pull request (PR) finally deletes the js files and the asset which had once been deleted with PR's #38823 and #39374 and then added back with PR #39431 for b/c with old layout overrides.

It is a simple revert of PR #39431 in the 5.0-dev branch plus adding the deleted compiled and minified js files from the media folder on update, i.e. adding them to the deleted files list in "script.php".

The deprecated language strings used by the removed js will be removed with PR #40296 .

Testing Instructions

Code review: Verify that this PR here is exactly reverting PR #40296 plus adding the deleted compiled and minified js files to the deleted files list in "script.php".

In addition, test an update from the latest 4.4-dev nightly build to the update package or custom URL created by Drone for this PR.

Actual result BEFORE applying this Pull Request

Deprecated js files and asset from PR #40296 are still present in the 5.0-dev branch.

Expected result AFTER applying this Pull Request

Deprecated js files and asset from PR #40296 are removed from the 5.0-dev branch, and the compiled and minified js files are deleted on update from 4.4-dev.

Link to documentations

Please select:

Delete the js files and the asset once deleted with PR's joomla#38823 and joomla#39374 and then added back with PR joomla#39431 for b/c reasons.
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.0-dev labels Apr 2, 2023
@richard67 richard67 mentioned this pull request Apr 2, 2023
4 tasks
@HLeithner
Copy link
Member

can you create a manual against joomla/Manual#28 please ?

@richard67
Copy link
Member Author

can you create a manual against joomla/Manual#28 please ?

@HLeithner If you tell me where to add that. I have no idea.

@HLeithner
Copy link
Member

@richard67
Copy link
Member Author

please add a section to this file https://github.com/joomla/Manual/blob/cf6b87662d3b086049c8b8078f8699db864a91dc/migrations/44-50/removed-backward-incompatibility.md

@HLeithner And how shall that section be titled and look like? Shall it be "JS removals" and be like the "CSS removals" section? What do you expect?

@HLeithner
Copy link
Member

@HLeithner And how shall that section be titled and look like? Shall it be "JS removals" and be like the "CSS removals" section? What do you expect?

I would expect something what you also find useful when you read this page ;-)

so something like the "CSS removals" would be a good start. including which webasset names are no longer available including the the path to the files and maybe the reason why it's no longer exists. (link to pr when it was removed the first time and that is is no longer used by joomla it self).

@richard67
Copy link
Member Author

@HLeithner What about the language strings removal in PR #40296 ? Does that need to be added to the developer manual, too?

@HLeithner
Copy link
Member

Actually yes, I added the deprecation for the language strings also the manual. So the removal have to be added too.

@richard67
Copy link
Member Author

Actually yes, I added the deprecation for the language strings also the manual. So the removal have to be added too.

@HLeithner But that belongs to @brianteeman 's PR #40296 , so I think he should do that. Otherwise I will need again instructions on where to do that.

@HLeithner
Copy link
Member

HLeithner commented Apr 2, 2023

already sent him a message

@richard67
Copy link
Member Author

@HLeithner For the javascript stuff see joomla/Manual#98 .

@HLeithner HLeithner added the Removal Removes functionality label Apr 7, 2023
@richard67 richard67 mentioned this pull request Apr 17, 2023
4 tasks
@HLeithner HLeithner merged commit d619c6c into joomla:5.0-dev Jun 26, 2023
@HLeithner HLeithner added this to the Joomla! 5.0 milestone Jun 26, 2023
@richard67 richard67 deleted the 5.0-dev-finally-remove-js-files-from-pr-39431 branch June 26, 2023 07:37
richard67 added a commit to richard67/joomla-cms that referenced this pull request Jul 3, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Jul 22, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Jul 23, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Jul 24, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Aug 12, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Aug 19, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Sep 1, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Sep 4, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Sep 4, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Sep 4, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Sep 5, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Sep 14, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Sep 19, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Sep 26, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Sep 26, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Sep 26, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Sep 26, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Oct 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NPM Resource Changed This Pull Request can't be tested by Patchtester Removal Removes functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants