Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants