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

Added ext-json to dependencies #1730

Merged
merged 1 commit into from
Apr 7, 2021
Merged

Added ext-json to dependencies #1730

merged 1 commit into from
Apr 7, 2021

Conversation

adamwojs
Copy link
Member

@adamwojs adamwojs commented Apr 7, 2021

Question Answer
Tickets -
Bug fix? yes
New feature? no
BC breaks? no
Tests pass? yes
Doc needed? no
License GPL-2.0

UDW, Sub-items list and config provider use json_* functions so ext-json should be declared as dependency.

Checklist:

  • Coding standards ($ composer fix-cs)
  • Ready for Code Review

@adamwojs adamwojs self-assigned this Apr 7, 2021
@lserwatka lserwatka merged commit 69f9a9f into 1.5 Apr 7, 2021
@lserwatka lserwatka deleted the missing_deps branch April 7, 2021 07:19
lucasOsti pushed a commit to lucasOsti/ezplatform-admin-ui that referenced this pull request Apr 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants