Skip to content

[nextjs] Fix pending redirects issues - #568

Merged
illiakovalenko merged 7 commits into
devfrom
bug/jss-10029-redirect-bugs
Jul 28, 2026
Merged

[nextjs] Fix pending redirects issues#568
illiakovalenko merged 7 commits into
devfrom
bug/jss-10029-redirect-bugs

Conversation

@art-alexeyenko

@art-alexeyenko art-alexeyenko commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

Description / Motivation

This is a composite PR to fix pending redirect proxy bugs, fresh off the heels of the refactor done recently.

  • Fixes locale always being attached on redirect when isLanguagePreserved is true, request URL does not have locale prefix and redirect does not change locale
  • Fixes Redirect Proxy not matching encoded URLs with unicode characters in Redirect Map

Testing Details

  • Unit Test Added
  • Manual Test/Other (Please elaborate)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

@changeset-bot

changeset-bot Bot commented Jul 27, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 0a88eaf

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@sitecore-content-sdk/nextjs Patch
@sitecore-content-sdk/content Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Comment thread packages/content/src/site/redirect-utils.ts Dismissed
@github-actions

github-actions Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

📦 Package Size and Test Coverage Report

Package Base Size PR Size Δ Change Base Coverage PR Coverage Δ Change
analytics-core 67.34 KB 67.34 KB ✅ 0.00 KB 97.40% 97.40% 0.00%
angular 314.76 KB 314.76 KB ✅ 0.00 KB 93.97% 94.04% +0.07%
cli 61.06 KB 61.06 KB ✅ 0.00 KB 71.36% 71.36% 0.00%
content 481.96 KB 481.96 KB ✅ 0.00 KB 92.86% 92.86% 0.00%
core 121.93 KB 121.93 KB ✅ 0.00 KB 91.09% 91.09% 0.00%
create-content-sdk-app 523.93 KB 523.93 KB ✅ 0.00 KB 96.47% 96.47% 0.00%
events 72.27 KB 72.27 KB ✅ 0.00 KB 97.97% 97.97% 0.00%
nextjs 543.01 KB 543.01 KB ✅ 0.00 KB 91.47% 91.47% 0.00%
personalize 64.01 KB 64.01 KB ✅ 0.00 KB 99.74% 99.74% 0.00%
react 235.00 KB 235.00 KB ✅ 0.00 KB 93.89% 93.89% 0.00%
search 8.25 KB 8.25 KB ✅ 0.00 KB 98.72% 98.72% 0.00%
Total ✅ 0.00 KB

@illiakovalenko illiakovalenko left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

👍

@illiakovalenko
illiakovalenko merged commit 6563736 into dev Jul 28, 2026
3 of 4 checks passed
@illiakovalenko
illiakovalenko deleted the bug/jss-10029-redirect-bugs branch July 28, 2026 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants