fix(clerk-js): Improve error message during Organization invitation - #1781
Conversation
🦋 Changeset detectedLatest commit: e8320a3 The changes in this PR will be included in the next version bump. This PR includes changesets to release 4 packages
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 |
fbc2146 to
20658da
Compare
20658da to
e8320a3
Compare
chanioxaris
left a comment
There was a problem hiding this comment.
Even though we improve the user feedback for this case, i am afraid this won't scale while our APIs will return additional error codes for more cases (e.g. email address exists in the blocklist). We have to think another way to fix that , probably a breaking one in order to fix this once and for all
|
As this is not breaking change, and enhances the current feedback we provide, I'll proceed with merging. We will continue discussing internally for a better solution that scales for many localized errors per FAPI endpoint |
|
This PR has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Description
This PR improves the localized message and also the implementation of the constructed message.
Checklist
npm testruns as expected.npm run buildruns as expected.Type of change
Packages affected
@clerk/clerk-js@clerk/clerk-react@clerk/nextjs@clerk/remix@clerk/types@clerk/themes@clerk/localizations@clerk/clerk-expo@clerk/backend@clerk/clerk-sdk-node@clerk/shared@clerk/fastify@clerk/chrome-extensiongatsby-plugin-clerkbuild/tooling/chore