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

IBX-2000: Added search into URL Wildcard tab #2042

Merged
merged 10 commits into from
May 25, 2022

Conversation

mateuszdebinski
Copy link
Contributor

@mateuszdebinski mateuszdebinski commented Apr 22, 2022

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

related to ezsystems/ezplatform-kernel#301

regression tests passed:
ibexa/commerce#88
ibexa/experience#39

Checklist:

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

src/lib/Form/Data/URLWildcard/URLWildcardListData.php Outdated Show resolved Hide resolved
src/lib/Form/Data/URLWildcard/URLWildcardListData.php Outdated Show resolved Hide resolved
src/lib/Form/Factory/FormFactory.php Outdated Show resolved Hide resolved
src/lib/Form/Type/URLWildcard/URLWildcardListType.php Outdated Show resolved Hide resolved
src/lib/Form/Type/URLWildcard/URLWildcardListType.php Outdated Show resolved Hide resolved
src/lib/Form/Type/URLWildcard/URLWildcardListType.php Outdated Show resolved Hide resolved
src/lib/Form/Type/URLWildcard/URLWildcardListType.php Outdated Show resolved Hide resolved
src/lib/Form/Type/URLWildcard/URLWildcardListType.php Outdated Show resolved Hide resolved
src/lib/Tab/URLManagement/URLWildcardsTab.php Outdated Show resolved Hide resolved
@adamwojs
Copy link
Member

Some of the comment are still is not addressed: #2042 (comment)

@mateuszdebinski
Copy link
Contributor Author

Some of the comment are still is not addressed: #2042 (comment)

@adamwojs now should be done

src/lib/Form/Type/URLWildcard/URLWildcardListType.php Outdated Show resolved Hide resolved
src/lib/Form/Type/URLWildcard/URLWildcardListType.php Outdated Show resolved Hide resolved
src/lib/Form/Type/URLWildcard/URLWildcardListType.php Outdated Show resolved Hide resolved
src/lib/Form/Type/URLWildcard/URLWildcardListType.php Outdated Show resolved Hide resolved
src/lib/Tab/URLManagement/URLWildcardsTab.php Outdated Show resolved Hide resolved
src/lib/Tab/URLManagement/URLWildcardsTab.php Outdated Show resolved Hide resolved
src/lib/Tab/URLManagement/URLWildcardsTab.php Outdated Show resolved Hide resolved
@sonarcloud
Copy link

sonarcloud bot commented May 25, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Contributor

@tomaszszopinski tomaszszopinski left a comment

Choose a reason for hiding this comment

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

QA approved on IbexaDXP 3.3 commerce.

@Nattfarinn Nattfarinn merged commit d382edf into 2.3 May 25, 2022
@Nattfarinn Nattfarinn deleted the IBX-2000_add_search_to_url_wildcard_tab branch May 25, 2022 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

10 participants