Skip to content

fix(nextjs): Use nextjs fetcher to identify internal page navigation - #3529

Merged
panteliselef merged 5 commits into
mainfrom
elef/sdk-1817-use-next-fetch-for-page-detection
Jun 6, 2024
Merged

fix(nextjs): Use nextjs fetcher to identify internal page navigation#3529
panteliselef merged 5 commits into
mainfrom
elef/sdk-1817-use-next-fetch-for-page-detection

Conversation

@panteliselef

@panteliselef panteliselef commented Jun 6, 2024

Copy link
Copy Markdown
Contributor

Description

This PR is enhancing the page detection mechanism for Nextjs.
It also fixes page detection for next@14.2.3

Checklist

  • npm test runs as expected.
  • npm run build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

@panteliselef panteliselef self-assigned this Jun 6, 2024
@changeset-bot

changeset-bot Bot commented Jun 6, 2024

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 9e43d82

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

This PR includes changesets to release 1 package
Name Type
@clerk/nextjs 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

@panteliselef
panteliselef marked this pull request as ready for review June 6, 2024 15:59

@brkalow brkalow left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👏

@panteliselef
panteliselef enabled auto-merge (squash) June 6, 2024 16:25
@panteliselef
panteliselef merged commit e2f1e8f into main Jun 6, 2024
@panteliselef
panteliselef deleted the elef/sdk-1817-use-next-fetch-for-page-detection branch June 6, 2024 16:41
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.

3 participants