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

V14: Fix redirect search #16285

Merged
merged 1 commit into from
May 15, 2024
Merged

V14: Fix redirect search #16285

merged 1 commit into from
May 15, 2024

Conversation

nikolajlauridsen
Copy link
Contributor

Fixes the GetAllRedirectUrlManagementController filter query parameter.

This didn't work because the SearchRedirectUrls taking skip/take didn't use the SearchRedirectUrls with a search term, but just called get all

Test

  1. Crate a child content node
  2. Rename the node to create a redirect.
  3. Ensure you can search for the redirect (the previous name)

@nikolajlauridsen nikolajlauridsen added the project/bellissima AKA "the new backoffice" label May 14, 2024
@Zeegaan Zeegaan added release/14.0.0-rc4 and removed project/bellissima AKA "the new backoffice" labels May 15, 2024
@Zeegaan
Copy link
Member

Zeegaan commented May 15, 2024

LGTM 🚀

@Zeegaan Zeegaan merged commit e46a8d2 into v14/dev May 15, 2024
16 checks passed
@Zeegaan Zeegaan deleted the v14/fix/fix-redirect-search branch May 15, 2024 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants