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

[4.1] skipto update #37227

Merged
merged 7 commits into from
Jun 12, 2022
Merged

[4.1] skipto update #37227

merged 7 commits into from
Jun 12, 2022

Conversation

brianteeman
Copy link
Contributor

Minor bugfix release upstream

Fixes problem in failure to create the SkipTo menu due to invalid configuration

Minor bugfix release upstream

Fixes problem in failure to create the SkipTo menu due to invalid configuration
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.1-dev labels Mar 8, 2022
@richard67
Copy link
Member

@brianteeman The build.js fails because it can't find a file, see here: https://ci.joomla.org/joomla/joomla-cms/52245/1/7 .

@brianteeman
Copy link
Contributor Author

brianteeman commented Mar 8, 2022

thanks for the heads up. I made a mistake upstream. I trusted someone elses work - oops

@brianteeman brianteeman deleted the skipto414 branch May 20, 2022 12:09
@brianteeman brianteeman restored the skipto414 branch June 8, 2022 13:45
@brianteeman brianteeman reopened this Jun 8, 2022
@roland-d roland-d changed the base branch from 4.1-dev to 4.2-dev June 12, 2022 18:44
@roland-d roland-d merged commit 896e0a0 into joomla:4.2-dev Jun 12, 2022
@roland-d
Copy link
Contributor

Thank you for the update

@roland-d roland-d added this to the Joomla 4.2.0 milestone Jun 12, 2022
@brianteeman
Copy link
Contributor Author

This should have been merged into 4.1.5 not 4.2

@brianteeman brianteeman deleted the skipto414 branch June 12, 2022 19:49
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