Skip to content

[5.2] Fix JS errors in admin menu#44410

Merged
pe7er merged 2 commits intojoomla:5.2-devfrom
C-Lodder:admin-menu-js-errors
Nov 7, 2024
Merged

[5.2] Fix JS errors in admin menu#44410
pe7er merged 2 commits intojoomla:5.2-devfrom
C-Lodder:admin-menu-js-errors

Conversation

@C-Lodder
Copy link
Member

@C-Lodder C-Lodder commented Nov 5, 2024

Pull Request for Issue #44408

Summary of Changes

This PR fixes the JS errors being thrown when creating an Admin Menu

Testing Instructions

See original issue

@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.2-dev labels Nov 5, 2024
@C-Lodder C-Lodder changed the title Fix JS errors in admin menu [5.2] Fix JS errors in admin menu Nov 5, 2024
@brianteeman
Copy link
Contributor

I have tested this item ✅ successfully on 8269f58

Thanks @C-Lodder


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

@Quy
Copy link
Contributor

Quy commented Nov 5, 2024

I have tested this item ✅ successfully on 8269f58


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

@Quy
Copy link
Contributor

Quy commented Nov 5, 2024

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Nov 5, 2024
@pe7er pe7er self-assigned this Nov 7, 2024
@pe7er pe7er enabled auto-merge (squash) November 7, 2024 10:28
@pe7er pe7er merged commit a79ec96 into joomla:5.2-dev Nov 7, 2024
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Nov 7, 2024
@Hackwar Hackwar added this to the Joomla! 5.2.1 milestone Nov 7, 2024
@Hackwar
Copy link
Member

Hackwar commented Nov 7, 2024

Thanks for your contribution!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants