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(nextjs): Drop deprecations #2006

Merged
merged 13 commits into from
Nov 3, 2023
Merged

Conversation

dimkl
Copy link
Contributor

@dimkl dimkl commented Nov 1, 2023

Description

Drop deprecations. Migration steps:

  • use NEXT_PUBLIC_CLERK_PUBLISHABLE_KEY instead of NEXT_PUBLIC_CLERK_FRONTEND_API env variable
  • use NEXT_PUBLIC_CLERK_JS_VERSION instead of CLERK_JS_VERSION
  • use CLERK_SECRET_KEY instead of CLERK_API_KEY
  • use publishableKey instead of frontendApi
  • use isEmailLinkError instead of isMagicLinkError
  • use EmailLinkErrorCode instead of MagicLinkErrorCode
  • use useEmailLink instead of useMagicLink
  • use @clerk/nextjs instead of @clerk/nextjs/app-beta
  • use @clerk/nextjs instead of @clerk/nextjs/ssr
  • use @clerk/nextjs instead of @clerk/nextjs/edge-middleware or @clerk/nextjs/edge-middlewarefiles
  • use middleware with authMiddleware instead of withClerkMiddleware
  • avoid using exported constants: API_URL, API_VERSION, CLERK_JS_URL, CLERK_JS_VERSION, DOMAIN, IS_SATELLITE, PROXY_URL, PUBLISHABLE_KEY, SECRET_KEY, SIGN_IN_URL, SIGN_UP_URL

Extra:

  • dropped @clerk/clerk-sdk-node dependency as there is no use anymore

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 1, 2023
Copy link

changeset-bot bot commented Nov 1, 2023

🦋 Changeset detected

Latest commit: a47baa2

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 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

@dimkl dimkl requested a review from a team November 1, 2023 15:46
@dimkl dimkl force-pushed the sdk-782-drop-deprecations-nextjs branch 2 times, most recently from 9c16e35 to a7c014f Compare November 1, 2023 17:26
@dimkl dimkl force-pushed the sdk-782-drop-deprecations-nextjs branch from a7c014f to a47baa2 Compare November 2, 2023 07:49
@dimkl dimkl mentioned this pull request Nov 3, 2023
24 tasks
Copy link
Member

@anagstef anagstef left a comment

Choose a reason for hiding this comment

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

💯

'@clerk/nextjs': major
---

Drop deprecations. Migration steps:
Copy link
Member

Choose a reason for hiding this comment

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

@dimkl Should we mark these as breaking?

Copy link
Contributor Author

@dimkl dimkl Nov 3, 2023

Choose a reason for hiding this comment

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

I thought that the major version bump should indicate that but we can also mark them as breaking changes. All the changes with major version are breaking. What do you think?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Merging this to continue with the cleanup and i will update the changeset if we want to somehow mark the changes as breaking.

@dimkl dimkl added this pull request to the merge queue Nov 3, 2023
Merged via the queue into main with commit f31031a Nov 3, 2023
7 checks passed
@dimkl dimkl deleted the sdk-782-drop-deprecations-nextjs branch November 3, 2023 09:00
@dimkl dimkl mentioned this pull request Nov 3, 2023
24 tasks
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.

3 participants