Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.3] Update deleted files in script.php for the upcoming 5.3.0-alpha1 #44450

Merged

Conversation

richard67
Copy link
Member

Pull Request for Issue # .

Summary of Changes

This pull request (PR) updates the list of files to be deleted on update in script.php to the latest changes in the 5.3-dev branch.

Currently there are only files from PR #44296 .

Testing Instructions

Code review and check the description of PR #44296 , or compare a recent 5.3-dev nightly build with a 5.2-dev nightly build and check for files which exist only in the 5.2-dev but not in 5.3-dev build.

Actual result BEFORE applying this Pull Request

The files added by this PR to the list are still there after an update from 5.2.x to 5.3-dev.

Expected result AFTER applying this Pull Request

The files added by this PR to the list are deleted after an update from 5.2.x to 5.3-dev.

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed

  • No documentation changes for manual.joomla.org needed

@richard67
Copy link
Member Author

richard67 commented Nov 13, 2024

@dgrammatiko As you have removed the build/media_source/system/scss/joomla-core-loader.scss from the sources with your PR #44296 : Can we safely delete the files on update? Or should we keep them for b/c reasons in case if a 3rd party extensions still uses them or refers to them?

@dgrammatiko
Copy link
Contributor

It's safe to remove those files as they where used exclusively during the bundling, they won't affect the web component as it already has those encapsulated (either before or after the PR #44296).
In sort, this is good, thanks

@richard67
Copy link
Member Author

@dgrammatiko Thanks for the quick reply.

@bembelimen bembelimen merged commit a7283c4 into joomla:5.3-dev Nov 16, 2024
3 checks passed
@bembelimen
Copy link
Contributor

Thx

@bembelimen bembelimen added this to the Joomla! 5.3.0 milestone Nov 16, 2024
@richard67 richard67 deleted the 5.3-dev-deleted-files-2024-11-13 branch November 16, 2024 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants