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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Peoplepicker fails without get tenant #3168

Closed
raleec opened this issue Apr 25, 2024 · 4 comments
Closed

Peoplepicker fails without get tenant #3168

raleec opened this issue Apr 25, 2024 · 4 comments
Labels
bug Something isn't working Needs: Triage 🔍

Comments

@raleec
Copy link

raleec commented Apr 25, 2024

Describe the bug
We currently use components of the @microsft/mgt NPM package. However, it is an old version 2.3.1.
I am upgrading from Angular 12 to Angular 15, which requires me to upgrade several other npm packages.
@microsft/mgt is the only one that is giving a hard time.
The peoplepicker should display users. However, the search option is now broken. I am getting a nasty error that is impossible to troubleshoot. "error":{"code":"OrganizationFromTenantGuidNotFound","message":"The tenant for tenant guid 'Tenant-in-Non-commercial-Cloud' does not exist."

To Reproduce
Steps to reproduce the behavior:

  1. Deploy application with peoplepicker in a non-commercial cloud
  2. Click on Peoplepicker component
  3. See error

Expected behavior
I'd like the people picker to display people from the Graph

Environment (please complete the following information):

  • OS: Windows
  • Browser edge
  • Framework Angular 15
  • Context Web
  • Version Latest
  • Provider unsure

Additional context
Add any other context about the problem here.

@raleec raleec added bug Something isn't working Needs: Triage 🔍 labels Apr 25, 2024
@gavinbarron
Copy link
Member

What version of the mgt libraries are you trying to upgrade to, and do you have a minimal reproduction you can share?

@raleec
Copy link
Author

raleec commented May 8, 2024

What version of the mgt libraries are you trying to upgrade to, and do you have a minimal reproduction you can share?

Attempting to upgrade to the latest version. Working on a minimal reproduction.

@raleec
Copy link
Author

raleec commented May 13, 2024

Upgrading to the latest version in the 2.x range versions: MGT-PEOPLE-PICKER/2.11.2, MgtMsal2Provider/2.11.2, mgt/2.11.2, graph-js/2.2.1 (featureUsage=6)

url: https://graph.microsoft.us/v1.0/me/people?$search=%22hen%22&$top=10&$filter=personType/class%20eq%20%27Person%27

@raleec
Copy link
Author

raleec commented May 14, 2024

I found the error, and it wasn't related to this failed call.
in short, the fix described here: #1471 created a condition where the people picker defaults to person. the existing code required group.

@raleec raleec closed this as completed May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Needs: Triage 🔍
Projects
Status: Done ✔️
Development

No branches or pull requests

2 participants