-
Notifications
You must be signed in to change notification settings - Fork 402
chore(clerk-js,types,localizations): Introduce minimum length for organization memberships search #5159
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
Conversation
🦋 Changeset detectedLatest commit: c264dc2 The changes in this PR will be included in the next version bump. This PR includes changesets to release 23 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 |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
905a136 to
89921a1
Compare
89921a1 to
8e3956a
Compare
8e3956a to
4ba3605
Compare
61cfb18 to
5f0b04c
Compare
c5df69c to
383e89e
Compare
52245bc to
e829bb9
Compare
d97775f to
4de61aa
Compare
1574aae to
80f74c9
Compare
75ef85c to
f83adf0
Compare
42f185d to
c6beeaa
Compare
| return ( | ||
| <Animated asChild> | ||
| <Flex sx={{ width: '100%' }}> | ||
| <Flex |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should follow up with descriptors for these elements. There currently is no way to target MemberSearchForm elements with the appearance prop.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| <Field.Root {...searchField}> | ||
| <InputWithIcon | ||
| value={value} | ||
| {...searchField.props} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😢 Ooff, thanks for catching that! I forgot to sanitize the input props... going to push a fix for that
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pushed a fix on 4a53af3 - we currently only rely on the blur + focus event handlers from the useFormControl context
8e0b49d to
c264dc2
Compare
|
I'll do another review through the changes next week, will avoid merging on a Friday |
|
We decided to park these changes for now in favor of not affecting the UX here due to this "jump" behavior of the input validation. FAPI PR has also been closed there we won't get a 4xx error even if the query gets sent with less than 3 characters. Let's keep analyzing the logs for the query performance, and decide later to react on the FAPI data layer instead of forcing the UI to not trigger the query. |


Description
useDebouncehook instead of relying onkeyupeventCleanShot.2025-02-13.at.20.31.57.mp4
Checklist
pnpm testruns as expected.pnpm buildruns as expected.Type of change