Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(types): Drop deprecations #2109

Merged
merged 6 commits into from
Nov 17, 2023
Merged

chore(types): Drop deprecations #2109

merged 6 commits into from
Nov 17, 2023

Conversation

dimkl
Copy link
Contributor

@dimkl dimkl commented Nov 10, 2023

Description

Drop deprecations. Migration steps:

  • drop orgs jwt claim from session token
  • change type of auth param of withServerAuth() callback to AuthObject from ServerSideAuth in gatsby-clerk-plugin
    • use auth.sessionClaims instead of auth.claims
    • use AuthObject properties from auth
  • use publishableKey instead of frontendApi
  • use ClerkProviderOptionsWrapper type instead of IsomorphicClerkOptions

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:

Packages affected

  • @clerk/backend
  • @clerk/chrome-extension
  • @clerk/clerk-js
  • @clerk/clerk-expo
  • @clerk/fastify
  • gatsby-plugin-clerk
  • @clerk/localizations
  • @clerk/nextjs
  • @clerk/clerk-react
  • @clerk/remix
  • @clerk/clerk-sdk-node
  • @clerk/shared
  • @clerk/themes
  • @clerk/types
  • build/tooling/chore

@dimkl dimkl self-assigned this Nov 10, 2023
@dimkl dimkl requested a review from a team as a code owner November 10, 2023 08:08
Copy link

changeset-bot bot commented Nov 10, 2023

🦋 Changeset detected

Latest commit: eb4a21e

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

This PR includes changesets to release 11 packages
Name Type
gatsby-plugin-clerk Major
@clerk/nextjs Major
@clerk/clerk-react Major
@clerk/remix Major
@clerk/types Major
@clerk/chrome-extension Patch
@clerk/clerk-expo Patch
@clerk/backend Patch
@clerk/clerk-js Patch
@clerk/fastify Patch
@clerk/clerk-sdk-node 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

.changeset/long-beds-mate.md Outdated Show resolved Hide resolved
@dimkl dimkl force-pushed the sdk-782-drop-deprecations-shared branch from fe363d9 to e6c457f Compare November 10, 2023 09:09
@dimkl dimkl force-pushed the sdk-782-drop-deprecations-types branch from 769e1c3 to 89d9c98 Compare November 10, 2023 09:12
packages/nextjs/src/types.ts Show resolved Hide resolved
packages/remix/src/client/types.ts Show resolved Hide resolved
@dimkl dimkl force-pushed the sdk-782-drop-deprecations-shared branch from e6c457f to cb0f046 Compare November 10, 2023 10:40
@dimkl dimkl force-pushed the sdk-782-drop-deprecations-types branch from 89d9c98 to 48c87b1 Compare November 10, 2023 10:47
@dimkl dimkl force-pushed the sdk-782-drop-deprecations-types branch 2 times, most recently from b05817b to 937338d Compare November 10, 2023 13:54
@dimkl dimkl force-pushed the sdk-782-drop-deprecations-shared branch from cb0f046 to 2970cba Compare November 13, 2023 15:53
@dimkl dimkl force-pushed the sdk-782-drop-deprecations-types branch from 937338d to e013129 Compare November 13, 2023 15:54
@dimkl dimkl force-pushed the sdk-782-drop-deprecations-shared branch 2 times, most recently from 4cbb4c9 to 55612d2 Compare November 13, 2023 18:32
.changeset/long-beds-mate.md Outdated Show resolved Hide resolved
@dimkl dimkl force-pushed the sdk-782-drop-deprecations-shared branch from 55612d2 to f052495 Compare November 15, 2023 13:45
Base automatically changed from sdk-782-drop-deprecations-shared to main November 15, 2023 13:58
@dimkl dimkl force-pushed the sdk-782-drop-deprecations-types branch from e013129 to fd01ed2 Compare November 15, 2023 14:27
@dimkl dimkl enabled auto-merge November 15, 2023 14:27
@dimkl dimkl disabled auto-merge November 15, 2023 14:27
@dimkl dimkl force-pushed the sdk-782-drop-deprecations-types branch from fd01ed2 to e0107dd Compare November 15, 2023 15:05
@dimkl dimkl enabled auto-merge November 15, 2023 15:05
@dimkl dimkl force-pushed the sdk-782-drop-deprecations-types branch from e0107dd to c82f3ae Compare November 15, 2023 15:14
@dimkl dimkl force-pushed the sdk-782-drop-deprecations-types branch 4 times, most recently from 2961583 to 6e7e3ec Compare November 15, 2023 23:49
@dimkl dimkl force-pushed the sdk-782-drop-deprecations-types branch from 6e7e3ec to acafd5b Compare November 16, 2023 17:23
@dimkl dimkl force-pushed the sdk-782-drop-deprecations-types branch from acafd5b to 418ddc0 Compare November 16, 2023 17:27
isSatellite: props.isSatellite || isTruthy(process.env.NEXT_PUBLIC_CLERK_IS_SATELLITE),
proxyUrl: props.proxyUrl || (process.env.NEXT_PUBLIC_CLERK_PROXY_URL as any), // added as any to cater issue with type resolution
domain: props.domain || (process.env.NEXT_PUBLIC_CLERK_DOMAIN as any), // added as any to cater issue with type resolution
isSatellite: props.isSatellite || (isTruthy(process.env.NEXT_PUBLIC_CLERK_IS_SATELLITE) as any), // added as any to cater issue with type resolution
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@panteliselef i had to do this in order for the MultidomainAndOrProxy types to pass. let's discuss it tomorrow and see if we can simplify the type or fix the current issues.

@dimkl dimkl force-pushed the sdk-782-drop-deprecations-types branch from 418ddc0 to d5e3377 Compare November 16, 2023 17:36
@dimkl
Copy link
Contributor Author

dimkl commented Nov 16, 2023

Moved localization related PRs to #2151

@dimkl dimkl force-pushed the sdk-782-drop-deprecations-types branch from cc78b25 to eb4a21e Compare November 17, 2023 15:22
@dimkl dimkl added this pull request to the merge queue Nov 17, 2023
Merged via the queue into main with commit 2a22aad Nov 17, 2023
7 checks passed
@dimkl dimkl deleted the sdk-782-drop-deprecations-types branch November 17, 2023 15:36
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.

4 participants