fix(UVE): UrlContentMap Redirection to Correct Page After Save or Publish #29896 #30334
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
zJaaal
approved these changes
Oct 15, 2024
|
fmontes
approved these changes
Oct 15, 2024
rjvelazco
approved these changes
Oct 15, 2024
valentinogiardino
deleted the
issue-29896-uve-fix-urlcontentmaps-redirect
branch
spbolton
pushed a commit
that referenced
this pull request
Nov 11, 2024
…lish #29896 (#30334) ### Proposed Changes * Refactored the logic to handle URL redirection after saving a UrlContentMap. * Added methods to extract and determine the target URL for redirection, ensuring the correct URL is used when publishing content. ### Checklist - [x] Tests - [x] Translations - [x] Security Implications Contemplated (add notes if applicable) ### Additional Info This fix addresses the issue where, after saving or publishing a UrlContentMap, the system incorrectly redirects to the content type’s detail page instead of the expected URL. The changes ensure that the correct URL is detected and used, preventing unwanted redirection after the save or publish action. ### Screenshots #### Before: https://github.com/user-attachments/assets/1b4f74fa-f0b7-46f4-b389-a1bc63f52f7b #### After: https://github.com/user-attachments/assets/84ea8aa1-a434-4c17-a4b3-b4f391a58d14
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed Changes
Checklist
Additional Info
This fix addresses the issue where, after saving or publishing a UrlContentMap, the system incorrectly redirects to the content type’s detail page instead of the expected URL. The changes ensure that the correct URL is detected and used, preventing unwanted redirection after the save or publish action.
Screenshots
Before:
issue-29896-uve-fix-urlcontentmaps-redirect.mov
After:
fix-issue-29896-uve-fix-urlcontentmaps-redirect.mov
This PR fixes: #29896