nextjs - ✅(Solved) Fix existing rewrite broken after version update from 15.5.6 and 16.0.0 [1 pull requests, 1 comments, 2 participants]

Official PRs (…)
ON THIS PAGE

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…
GitHub stats
vercel/next.js#86385Fetched 2026-04-08 02:11:17
View on GitHub
Comments
1
Participants
2
Timeline
5
Reactions
1
Timeline (top)
commented ×1cross-referenced ×1issue_type_added ×1labeled ×1

Fix Action

Fixed

PR fix notes

PR #1594: MDZ-1577: New CRM Field Account Closing Date

Description (problem / solution / changelog)

Midaz Pull Request Checklist

Pull Request Type

  • Console
  • Documentation
  • Infra
  • Mdz
  • Onboarding
  • Pipeline
  • Transaction
  • Tests
  • CRM

Checklist

Please check each item after it's completed.

  • I have tested these changes locally.
  • I have updated the documentation accordingly.
  • I have added necessary comments to the code, especially in complex areas.
  • I have ensured that my changes adhere to the project's coding standards.
  • I have checked for any potential security issues.
  • I have ensured that all tests pass.
  • I have updated the version appropriately (if applicable).
  • I have confirmed this code is ready for review.

Obs: Please, always remember to target your PR to develop branch instead of main.

Additional Notes

  • Add new field closingDate to Alias model
  • Validation for closingDate < createdAt
<!-- This is an auto-generated comment: release notes by coderabbit.ai -->

Summary by CodeRabbit

  • New Features

    • Added a Closing Date field for aliases, persisted to storage, plus server-side validation enforcing it is not before creation and surfaced as a structured business error.
  • Bug Fixes

    • Deletion now no-ops when an alias has no related holder links and treats that case as successful.
  • Tests

    • Added/expanded tests for closing-date validation, alias enrichment with link types, and updated alias-deletion behavior.

<sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub>

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Changed files

  • CHANGELOG.md (modified, +11/-0)
  • components/console/package-lock.json (modified, +46/-46)
  • components/console/package.json (modified, +3/-3)
  • components/crm/internal/adapters/mongodb/alias/alias.go (modified, +22/-19)
  • components/crm/internal/services/delete-alias.go (modified, +8/-6)
  • components/crm/internal/services/delete-alias_test.go (modified, +5/-3)
  • components/crm/internal/services/enrich-alias-with-link-type_test.go (added, +212/-0)
  • components/crm/internal/services/update-alias.go (modified, +19/-6)
  • components/crm/internal/services/validate-alias-closing-date.go (added, +41/-0)
  • components/crm/internal/services/validate-alias-closing-date_test.go (added, +170/-0)
  • pkg/constant/errors.go (modified, +11/-10)
  • pkg/errors.go (modified, +6/-0)
  • pkg/mmodel/alias.go (modified, +3/-0)

Code Example

bun dev

---

Binaries:
  node: 24.0.2
Relevant packages:
  next: 16.0.3
RAW_BUFFERClick to expand / collapse

Link to the code that reproduces this issue

https://github.com/gabrielmfern/rewrite-broken-in-version-update

To Reproduce

  1. Run dev server
    bun dev
  2. Open http://localhost:3000
  3. Click the link and see that there's no navigation

Current vs. Expected behavior

I would expect navigation to properly work and navigate to /docs instead of doing nothing.

Provide environment information

Binaries:
  node: 24.0.2
Relevant packages:
  next: 16.0.3

Which area(s) are affected? (Select all that apply)

Redirects

Which stage(s) are affected? (Select all that apply)

Vercel (Deployed), next start (local), next build (local), next dev (local)

Additional context

I was able to narrow the issue down to 15.6.0-canary.48 onwards, with 15.6.0-canary.47 working just fine.

extent analysis

TL;DR

  • The issue can likely be resolved by downgrading the Next.js version to a stable release before 15.6.0-canary.48, as the problem seems to be introduced in this version.

Guidance

  • Verify the issue by testing with 15.6.0-canary.47 to confirm that it's indeed a regression introduced after this version.
  • Check the release notes and changelogs of Next.js versions after 15.6.0-canary.47 to see if there are any known issues or breaking changes related to redirects.
  • Test the application with a stable version of Next.js (e.g., 15.5.x) to see if the navigation works as expected.
  • If downgrading is not feasible, consider opening an issue on the Next.js GitHub repository to report the bug and ask for assistance.

Example

No code snippet is provided as the issue seems to be related to a specific version of Next.js rather than a code-specific problem.

Notes

  • The solution assumes that the issue is indeed caused by a regression in Next.js and not by a change in the application code.
  • Downgrading Next.js might introduce other compatibility issues, so it's essential to test the application thoroughly after making this change.

Recommendation

  • Apply workaround: Downgrade Next.js to a version before 15.6.0-canary.48 (e.g., 15.6.0-canary.47 or a stable 15.5.x release) to avoid the regression until a fixed version is available.

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING

nextjs - ✅(Solved) Fix existing rewrite broken after version update from 15.5.6 and 16.0.0 [1 pull requests, 1 comments, 2 participants]