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

EZP-31059: Limited URL Search Results to unique entries only #2876

Merged
merged 6 commits into from
Dec 5, 2019
Merged

Conversation

adamwojs
Copy link
Member

@adamwojs adamwojs commented Nov 15, 2019

Question Answer
JIRA issue EZP-31059
Bug/Improvement yes
New feature no
Target version master
BC breaks no
Tests pass yes
Doc needed no

Failed build: https://travis-ci.org/ezsystems/ezpublish-kernel/jobs/612484652?utm_medium=notification&utm_source=github_status

TODO:

  • Implement feature / fix a bug.
  • Implement tests.
  • Fix new code according to Coding Standards ($ composer fix-cs).
  • Ask for Code Review.

@adamwojs adamwojs added the Bug label Nov 15, 2019
@adamwojs adamwojs self-assigned this Nov 15, 2019
@adamwojs adamwojs changed the title EZP-31059: Limited URL Search Results to unique entries only (tests) EZP-31059: Limit URL Search Results to unique entries only Nov 15, 2019
@adamwojs adamwojs changed the base branch from master to 7.5 November 15, 2019 17:39
@adamwojs adamwojs requested review from alongosz, mikadamczyk, vidarl and ViniTou and removed request for vidarl November 19, 2019 10:49
@adamwojs
Copy link
Member Author

PR updated according to @mikadamczyk code review suggestions.

@alongosz alongosz changed the title EZP-31059: Limit URL Search Results to unique entries only EZP-31059: Limited URL Search Results to unique entries only Dec 2, 2019
@katarzynazawada katarzynazawada self-assigned this Dec 5, 2019
@alongosz alongosz merged commit 05f3918 into 7.5 Dec 5, 2019
@alongosz alongosz deleted the ezp_31059 branch December 5, 2019 13:00
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.

4 participants