Skip to content

PeoplePicker: Removed noResultsFound error for no user input#3431

Merged
musale merged 2 commits intomicrosoftgraph:mainfrom
Kanupriya1511:kanupriya/peoplePicker_noMatchesError
Apr 9, 2025
Merged

PeoplePicker: Removed noResultsFound error for no user input#3431
musale merged 2 commits intomicrosoftgraph:mainfrom
Kanupriya1511:kanupriya/peoplePicker_noMatchesError

Conversation

@Kanupriya1511
Copy link
Copy Markdown
Contributor

Closes #3430

PR Type

Bugfix

Description of the changes

Added check in people picker to see if there's no input from the user, in which case the error tooltip: We didn't find any matches won't be shown

PR checklist

  • Project builds (yarn build) and changes have been tested in at least two supported browsers (Edge + non-Chromium based browser)
  • All public APIs (classes, methods, etc) have been documented following the jsdoc syntax
  • Stories have been added and existing stories have been tested
  • Added appropriate documentation. Docs PR:
  • License header has been added to all new source files (yarn setLicense)
  • Contains NO breaking changes

Other information

@Kanupriya1511 Kanupriya1511 requested a review from a team as a code owner April 3, 2025 10:37
@microsoft-github-policy-service
Copy link
Copy Markdown
Contributor

Thank you for creating a Pull Request @@Kanupriya1511.

This is a checklist for the PR reviewer(s) to complete before approving and merging this PR:

  • I have verified a documentation PR has been linked and is approved (or not applicable)
  • I have ran this PR locally and have tested the fix/feature
  • I have verified that stories have been added to storybook (or not applicable)
  • I have tested existing stories in storybook to verify no regression has occured
  • I have tested the solution in at least two browsers (Edge + 1 non-Chromium based browser)

@Kanupriya1511 Kanupriya1511 changed the title Removed noResultsFound error for no user input PeoplePicker: Removed noResultsFound error for no user input Apr 3, 2025
@github-project-automation github-project-automation bot moved this to Done ✔️ in Graph Toolkit Apr 9, 2025
@musale musale merged commit 50314d7 into microsoftgraph:main Apr 9, 2025
9 of 10 checks passed
@github-project-automation github-project-automation bot moved this from Done ✔️ to Shipped 🚀 in Graph Toolkit Apr 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

[BUG] People picker incorrectly displays error message with no input

3 participants