Skip to content

chore(astro,clerk-js): Inject windowNavigate through router functions - #3922

Merged
panteliselef merged 3 commits into
mainfrom
elef/eco-50-refactor-duplicate-windownavigate-from-astro
Aug 13, 2024
Merged

chore(astro,clerk-js): Inject windowNavigate through router functions#3922
panteliselef merged 3 commits into
mainfrom
elef/eco-50-refactor-duplicate-windownavigate-from-astro

Conversation

@panteliselef

Copy link
Copy Markdown
Contributor

Description

In some cases (like in Astro) we want to reconstruct the original windowNavigate functionality, that is because the framework might not have a router.

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:

In some cases (like in Astro) we want to reconstruct the original windowNavigate functionality, that is because the framework might not have a router.
@panteliselefpanteliselef self-assigned this Aug 8, 2024
@changeset-bot

changeset-botBot commented Aug 8, 2024

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 3d9c1b3

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

This PR includes changesets to release 18 packages
NameType
@clerk/clerk-jsMinor
@clerk/astroMinor
@clerk/typesMinor
@clerk/chrome-extensionPatch
@clerk/clerk-expoPatch
@clerk/backendPatch
@clerk/elementsPatch
@clerk/expressPatch
@clerk/fastifyPatch
@clerk/localizationsPatch
@clerk/nextjsPatch
@clerk/clerk-reactPatch
@clerk/remixPatch
@clerk/clerk-sdk-nodePatch
@clerk/sharedPatch
@clerk/tanstack-startPatch
@clerk/testingPatch
@clerk/themesPatch

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

…ndownavigate-from-astro
# Conflicts:
#	packages/astro/src/internal/create-clerk-instance.ts
@panteliselef
panteliselef enabled auto-merge (squash) August 13, 2024 18:06
@panteliselef
panteliselef merged commit dc0e1c3 into mainAug 13, 2024
@panteliselef
panteliselef deleted the elef/eco-50-refactor-duplicate-windownavigate-from-astro branch August 13, 2024 18:07
Sign up for freeto 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.

5 participants

@panteliselef@tmilewski@octoper@anagstef@clerk-cookie