NPM Resource Changed PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar LadySolveig
LadySolveig
22 Sep 2025

Pull Request for Issue # .

Summary of Changes

Alternative to #46146

This PR is based on #46146 and adds running the versioning script on every build with npm install or npm ci

Testing Instructions

WIP

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar LadySolveig LadySolveig - open - 22 Sep 2025
avatar LadySolveig LadySolveig - change - 22 Sep 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 22 Sep 2025
Category JavaScript Repository NPM Change
avatar LadySolveig LadySolveig - change - 22 Sep 2025
The description was changed
avatar LadySolveig LadySolveig - edited - 22 Sep 2025
avatar HLeithner
HLeithner - comment - 15 Oct 2025

This pull request has been automatically rebased to 5.4-dev.

avatar brianteeman
brianteeman - comment - 3 Nov 2025

as #46146 was merged is there anything to do here?

avatar richard67
richard67 - comment - 3 Nov 2025

@brianteeman Not really something to do here. @LadySolveig wanted to keep it open as a reminder for later discussions.

avatar HLeithner HLeithner - change - 24 Nov 2025
Title
[5.3] Fix missing copyright in media assets + always run versioning on media asset build
[5.4] Fix missing copyright in media assets + always run versioning on media asset build
avatar HLeithner HLeithner - edited - 24 Nov 2025
avatar github-actions[bot]
github-actions[bot] - comment - 28 Feb 2026

This pull request has conflicts, please resolve those before we can evaluate the pull request.

avatar richard67 richard67 - change - 28 Feb 2026
Labels Added: Conflicting Files NPM Resource Changed PR-5.4-dev
avatar richard67 richard67 - change - 28 Feb 2026
Labels Removed: Conflicting Files
avatar richard67
richard67 - comment - 28 Feb 2026

@LadySolveig Even if it's only a draft and a reminder, I've allowed myself to solve the merged conflicts due to your other merged PR #46146 with the alternative approach. This was necessary for later being able to rebase the PR to 6.1-dev. The changed files should be the same as before, only that the later change from your PR #46609 is also included now. If that is wrong, please revert that.

avatar LadySolveig LadySolveig - change - 1 Mar 2026
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2026-03-01 20:43:10
Closed_By LadySolveig
avatar LadySolveig LadySolveig - close - 1 Mar 2026
avatar LadySolveig
LadySolveig - comment - 1 Mar 2026

Thank you Richard, since I hope this one #46879 will be merged, I will close for now.
I should have done that earlier. I hope you didn't have too much work to do with this.
If there is still a need for it afterwards, we can revisit, but I did not get the impression that there was much interest.

Add a Comment

Login with GitHub to post a comment