Skip to content

chore(clerk-js,types,backend): Remove MemberRole Type#2388

Merged
panteliselef merged 2 commits into
mainfrom
elef/core-940-remove-memberrole-type
Dec 19, 2023
Merged

chore(clerk-js,types,backend): Remove MemberRole Type#2388
panteliselef merged 2 commits into
mainfrom
elef/core-940-remove-memberrole-type

Conversation

@panteliselef

Copy link
Copy Markdown
Contributor

Description

MemberRole would always include the old role keys admin, member, guest_member. If developers still depend on them after the introduction of custom roles, the can provide them as their custom types for authorization.

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

@panteliselef panteliselef self-assigned this Dec 18, 2023
@changeset-bot

changeset-bot Bot commented Dec 18, 2023

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 120f770

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

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

`MemberRole` would always include the old role keys `admin`, `member`, `guest_member`. If developers still depend on them after the introduction of custom roles, the can provide them as their custom types for authorization.
@panteliselef panteliselef force-pushed the elef/core-940-remove-memberrole-type branch from 84e4626 to 14bf26f Compare December 18, 2023 13:01
@panteliselef panteliselef requested a review from a team December 18, 2023 14:50
Comment thread .changeset/orange-files-end.md
@panteliselef panteliselef added this pull request to the merge queue Dec 18, 2023
@panteliselef panteliselef removed this pull request from the merge queue due to a manual request Dec 18, 2023
@panteliselef panteliselef added this pull request to the merge queue Dec 19, 2023
Merged via the queue into main with commit fe2607b Dec 19, 2023
@panteliselef panteliselef deleted the elef/core-940-remove-memberrole-type branch December 19, 2023 08:52
dimkl added a commit to clerk/clerk-docs that referenced this pull request Jan 13, 2024
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.

3 participants