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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(*): Improve @clerk/backend DX [Part 5 - Drop api/resources from exports] #2365

Merged
merged 1 commit into from
Dec 15, 2023

Conversation

dimkl
Copy link
Member

@dimkl dimkl commented Dec 14, 2023

Description

Drop the api/resources related classes (kept only types of User, Session, Organization, WebhookEvent, WebhookEventType) since there are considered internal and there shouldn't be used.
This change will reduce the public API of the @clerk/backend and all dependents packages.

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

Copy link

changeset-bot bot commented Dec 14, 2023

馃 Changeset detected

Latest commit: 0b2da88

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

This PR includes changesets to release 6 packages
Name Type
gatsby-plugin-clerk Major
@clerk/clerk-sdk-node Major
@clerk/backend Major
@clerk/fastify Major
@clerk/nextjs Major
@clerk/remix 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 force-pushed the sdk-974-improve-dx-backend-p4-jwt branch from 88b5f26 to 2cafe50 Compare December 15, 2023 00:19
@dimkl dimkl force-pushed the sdk-974-improve-dx-backend-p5-cleanup branch from 10a7198 to 7a299f3 Compare December 15, 2023 00:24
@dimkl dimkl force-pushed the sdk-974-improve-dx-backend-p4-jwt branch 3 times, most recently from 4ee2894 to 6570b0f Compare December 15, 2023 12:17
Base automatically changed from sdk-974-improve-dx-backend-p4-jwt to main December 15, 2023 13:41
@dimkl dimkl force-pushed the sdk-974-improve-dx-backend-p5-cleanup branch 2 times, most recently from 1ee0cf1 to d0a896d Compare December 15, 2023 14:05
@dimkl dimkl force-pushed the sdk-974-improve-dx-backend-p5-cleanup branch from d0a896d to 0b2da88 Compare December 15, 2023 14:10
@SokratisVidros SokratisVidros merged commit 1ad910e into main Dec 15, 2023
6 of 8 checks passed
@SokratisVidros SokratisVidros deleted the sdk-974-improve-dx-backend-p5-cleanup branch December 15, 2023 14:17
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.

None yet

3 participants