Skip to content

chore(fastify): Drop deprecations #1976

Merged
dimkl merged 2 commits into
mainfrom
sdk-782-drop-deprecations-fastify
Nov 1, 2023
Merged

chore(fastify): Drop deprecations #1976
dimkl merged 2 commits into
mainfrom
sdk-782-drop-deprecations-fastify

Conversation

@dimkl

@dimkl dimkl commented Oct 30, 2023

Copy link
Copy Markdown
Contributor

Description

Drop deprecations. Migration steps:

  • use CLERK_SECRET_KEY instead of CLERK_API_KEY env variable
  • use secretKey instead of apiKey
  • use CLERK_PUBLISHABLE_KEY instead of CLERK_FRONTEND_API env variable
  • use publishableKey instead of frontendApi

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 Oct 30, 2023
@changeset-bot

changeset-bot Bot commented Oct 30, 2023

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 385e8e2

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

This PR includes changesets to release 1 package
Name Type
@clerk/fastify Major

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

frontendApi: constants.FRONTEND_API,
});
// TODO(@dimkl): use empty string for frontendApi until type is fixed in @clerk/backend to drop it
const interstitialHtmlPage = clerkClient.localInterstitial({ publishableKey, frontendApi: '' });

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

❓ oh is this required ? 😅

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yes. Will be fixed in the @clerk/backend deprecations PR

@dimkl

dimkl commented Oct 31, 2023

Copy link
Copy Markdown
Contributor Author

I will close this tomorrow as part of testing the alpha-v5 tags after merging #1970

@dimkl dimkl closed this Oct 31, 2023
@dimkl dimkl reopened this Oct 31, 2023
@dimkl
dimkl force-pushed the sdk-782-drop-deprecations-fastify branch from 769dbf5 to 1a366d5 Compare October 31, 2023 21:16
@dimkl dimkl changed the title chore(fastify): Drop fastify deprecations chore(fastify): Drop deprecations Oct 31, 2023
@dimkl
dimkl force-pushed the sdk-782-drop-deprecations-fastify branch from 1a366d5 to 385e8e2 Compare November 1, 2023 09:22
@dimkl
dimkl added this pull request to the merge queue Nov 1, 2023
Merged via the queue into main with commit cfb50e1 Nov 1, 2023
@dimkl
dimkl deleted the sdk-782-drop-deprecations-fastify branch November 1, 2023 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants