Skip to content

[4.0] Updating dependencies for PHP and JS#30248

Closed
Hackwar wants to merge 2 commits intojoomla:4.0-devfrom
Hackwar:j4dependencies
Closed

[4.0] Updating dependencies for PHP and JS#30248
Hackwar wants to merge 2 commits intojoomla:4.0-devfrom
Hackwar:j4dependencies

Conversation

@Hackwar
Copy link
Copy Markdown
Member

@Hackwar Hackwar commented Jul 31, 2020

This does what the title says.

@joomla-cms-bot joomla-cms-bot added Composer Dependency Changed NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev labels Jul 31, 2020
"@webcomponents/webcomponentsjs": "^2.4.4",
"accessibility": "^3.0.9",
"awesomplete": "1.1.5",
"bootstrap": "^4.5.0",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Upgrade to 4.5.2?

@brianteeman
Copy link
Copy Markdown
Contributor

this PR is already out of date

@richard67 richard67 mentioned this pull request Aug 8, 2020
@richard67
Copy link
Copy Markdown
Member

@Hackwar Could you check and if necessary update for meanwhile never versions where possible?

And could you include an update of bootstrap 4 from the 4.5.0 we have now to the recent 4.5.2?

The latter would solve issue #30319 .

If you don't have time I can try to help with a PR to your branch, at least for the bootstrap stuff. Just ping me if necessary.

@richard67
Copy link
Copy Markdown
Member

This PR is meanwhile out of date, but there is a new one: #30334 , which also includes bootstrap 4 update.

Closing in favour of #30334 . Sorry @Hackwar that your PR had to wait to long for tests that it became outdated.

@richard67 richard67 closed this Aug 10, 2020
@Hackwar Hackwar deleted the j4dependencies branch May 14, 2021 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Composer Dependency Changed 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.

5 participants