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.0] Realistic test for import maps #41018

Closed
wants to merge 13 commits into from

Conversation

dgrammatiko
Copy link
Contributor

THIS SHOULD NOT BE MERGED. IT'S JUST FOR DEMOING THE IMPORT MAPS

Summary of Changes

Check the showon script: it's split to 2 files, the class and the initiator

Testing Instructions

Open the global config and check that the Site Offline still works

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Screenshot 2023-06-22 at 22 39 29 Screenshot 2023-06-22 at 22 40 16

@HLeithner

@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.0-dev labels Jun 22, 2023
@dgrammatiko dgrammatiko mentioned this pull request Jun 22, 2023
3 tasks
@HLeithner
Copy link
Member

@HLeithner HLeithner marked this pull request as draft June 22, 2023 21:07
@HLeithner
Copy link
Member

thanks

@HLeithner
Copy link
Member

can you please resolve the conflicts #40714 has been merged

@dgrammatiko
Copy link
Contributor Author

Replaced by #41034

@dgrammatiko dgrammatiko deleted the esm-iimportmap branch June 24, 2023 10:14
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.

4 participants