microsoft-graph-explorer-v4 icon indicating copy to clipboard operation
microsoft-graph-explorer-v4 copied to clipboard

Tenant name does not appear when signing in as a guest user

Open darrelmiller opened this issue 3 years ago • 8 comments

Steps to reproduce

  • Access Graph explorer using tenanted URL e.g. https://developer.microsoft-tst.com/en-us/graph/graph-explorer?tenant=tavis.ca

  • Sign in as a guest user to the tenant

Expected

To see the tenant name in the header and in the user profile card.

Actual

image

darrelmiller avatar Jun 17 '22 16:06 darrelmiller

Hey @darrelmiller, I am trying to reproduce this though I don't get an option to sign in as a guest user. How can I achieve this?

ElinorW avatar Jun 20 '22 09:06 ElinorW

@ElinorW You have to add the tenant query parameter to the URL e.g.

https://developer.microsoft-tst.com/en-us/graph/graph-explorer?tenant=tavis.ca

In order to test it you will need to go into a dev tenant and then add your microsoft account as a guest in that tenant. Then you specify the dev tenant domain name in the tenant query parameter and sign in as a guest user.

I had completely forgot that this feature had been implemented.

darrelmiller avatar Jun 20 '22 12:06 darrelmiller

Got it, I have been able to reproduce this now.

So, the first time I ran this, it was blank for me too... then after testing again, I got some tenant information image

could you try running it again and see if you'll get anything this time.

ElinorW avatar Jun 20 '22 13:06 ElinorW

@ElinorW I noticed that sometimes it just takes a bit of time to load. I think this is worth investigating further

adhiambovivian avatar Jun 21 '22 14:06 adhiambovivian

@adhiambovivian Could be because I have to make a separate network call to get tenant info... This info doesn't seem to be present on a user's account data

ElinorW avatar Jun 22 '22 15:06 ElinorW

This issue has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 4 days. It will be closed if no further activity occurs within 3 days of this comment.

ghost avatar Jun 26 '22 16:06 ghost

@ElinorW is this really blocked or work we haven't gotten to yet.

ddyett avatar Aug 02 '22 19:08 ddyett

@ddyett this isn't blocked, I have moved it to the backlog and part of the the August milestone

ElinorW avatar Aug 03 '22 11:08 ElinorW

This is resolved. For the tenant value I had to use a tenant ID as opposed to the tenant name

https://user-images.githubusercontent.com/18407044/184650471-ca6dd384-f89e-49e3-b415-f2d48695fd47.mp4

ElinorW avatar Aug 15 '22 13:08 ElinorW

This has been resolved also when with the use of a tenant name.

ElinorW avatar Aug 19 '22 19:08 ElinorW