Skip to content

chore(types): Drop deprecations - #2109

Merged
dimkl merged 6 commits into
mainfrom
sdk-782-drop-deprecations-types
Nov 17, 2023
Merged

chore(types): Drop deprecations#2109
dimkl merged 6 commits into
mainfrom
sdk-782-drop-deprecations-types

Conversation

@dimkl

@dimkldimkl commented Nov 10, 2023

Copy link
Copy Markdown
Contributor

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

@dimkldimkl self-assigned this Nov 10, 2023
@dimkl
dimkl requested a review from a team as a code ownerNovember 10, 2023 08:08
@changeset-bot

changeset-botBot commented Nov 10, 2023

Copy link
Copy Markdown

🦋 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
NameType
gatsby-plugin-clerkMajor
@clerk/nextjsMajor
@clerk/clerk-reactMajor
@clerk/remixMajor
@clerk/typesMajor
@clerk/chrome-extensionPatch
@clerk/clerk-expoPatch
@clerk/backendPatch
@clerk/clerk-jsPatch
@clerk/fastifyPatch
@clerk/clerk-sdk-nodePatch

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.changeset/long-beds-mate.md Outdated
@dimkl
dimklforce-pushed the sdk-782-drop-deprecations-shared branch from fe363d9 to e6c457fCompareNovember 10, 2023 09:09
@dimkl
dimklforce-pushed the sdk-782-drop-deprecations-types branch from 769e1c3 to 89d9c98CompareNovember 10, 2023 09:12
Comment threadpackages/nextjs/src/types.ts
Comment threadpackages/remix/src/client/types.ts
@dimkl
dimklforce-pushed the sdk-782-drop-deprecations-shared branch from e6c457f to cb0f046CompareNovember 10, 2023 10:40
@dimkl
dimklforce-pushed the sdk-782-drop-deprecations-types branch from 89d9c98 to 48c87b1CompareNovember 10, 2023 10:47
@dimkl
dimklforce-pushed the sdk-782-drop-deprecations-types branch 2 times, most recently from b05817b to 937338dCompareNovember 10, 2023 13:54
@dimkl
dimklforce-pushed the sdk-782-drop-deprecations-shared branch from cb0f046 to 2970cbaCompareNovember 13, 2023 15:53
@dimkl
dimklforce-pushed the sdk-782-drop-deprecations-types branch from 937338d to e013129CompareNovember 13, 2023 15:54
@dimkl
dimklforce-pushed the sdk-782-drop-deprecations-shared branch 2 times, most recently from 4cbb4c9 to 55612d2CompareNovember 13, 2023 18:32
Comment thread.changeset/long-beds-mate.md Outdated
@dimkl
dimklforce-pushed the sdk-782-drop-deprecations-shared branch from 55612d2 to f052495CompareNovember 15, 2023 13:45
Base automatically changed from sdk-782-drop-deprecations-shared to mainNovember 15, 2023 13:58
@dimkl
dimklforce-pushed the sdk-782-drop-deprecations-types branch from e013129 to fd01ed2CompareNovember 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
dimklforce-pushed the sdk-782-drop-deprecations-types branch from fd01ed2 to e0107ddCompareNovember 15, 2023 15:05
@dimkl
dimkl enabled auto-merge November 15, 2023 15:05
@dimkl
dimklforce-pushed the sdk-782-drop-deprecations-types branch from e0107dd to c82f3aeCompareNovember 15, 2023 15:14
@dimkl
dimklforce-pushed the sdk-782-drop-deprecations-types branch 4 times, most recently from 2961583 to 6e7e3ecCompareNovember 15, 2023 23:49
@dimkl
dimklforce-pushed the sdk-782-drop-deprecations-types branch from 6e7e3ec to acafd5bCompareNovember 16, 2023 17:23
@dimkl
dimklforce-pushed the sdk-782-drop-deprecations-types branch from acafd5b to 418ddc0CompareNovember 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
Copy Markdown
ContributorAuthor

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
dimklforce-pushed the sdk-782-drop-deprecations-types branch from 418ddc0 to d5e3377CompareNovember 16, 2023 17:36
@dimkl

Copy link
Copy Markdown
ContributorAuthor

Moved localization related PRs to #2151

@dimkl
dimklforce-pushed the sdk-782-drop-deprecations-types branch from cc78b25 to eb4a21eCompareNovember 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 2a22aadNov 17, 2023
@dimkl
dimkl deleted the sdk-782-drop-deprecations-types branch November 17, 2023 15:36
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.

4 participants

@dimkl@nikosdouvlis@panteliselef@clerk-cookie