search

MicrosoftDocs / azure-docs

Open source documentation of Microsoft Azure
https://docs.microsoft.com/azure
Creative Commons Attribution 4.0 International
10.03k stars 21.05k forks source link

Minor issues with instructions for the 'Share using RBAC' section #122589

Closed sabuncumurat closed 1 week ago

sabuncumurat commented 2 weeks ago

Although this is minor, I am going to report it anyways, because it is somewhat confusing:

In the 'Portal' instructions for the section Share using RBAC:

Bullet 2 says:

  1. Under Add a role assignment, select Add. The Add a role assignment pane will open.

Based on the actual UI experience, it should say:

2. Under Add, select Add role assignment. The Add role assignment page will open.

Bullet 4 says:

  1. Under assign access to, leave the default of Microsoft Entra user, group, or service principal.

First, it needs to be mentioned that the user needs to be in the Members tab. Then it should be worded as:

4. For Assign access to, keep the default of User, group, or service principal.

Bullet 5 says:

  1. Under Select, type in the email address of the person that you would like to invite.

Should be reworded as:

5. Click Select members and choose a user account from the pane that opens on the right.

Note that there is no invite facility at this point. Either invite happens earlier (which is how I got my sample to work), or there is more to this than I know. But I am positive that I do not get an invite flow at this step. If the invite mechanism is indeed not present at this step, then the next step needs to be rewritten as well.

I know the UI changes frequently, so please accept the above as constructive comments! :)

Thanks.

Document Details

⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

PesalaPavan commented 2 weeks ago

@sabuncumurat Thanks for your feedback! We will investigate and update as appropriate.

PesalaPavan commented 1 week ago

@sabuncumurat Thank you for bringing this to our attention! We've initiated a pull request (PR) to address this issue. Once the author reviews the changes, they will be merged, and the updates should go live shortly. Your diligence is much appreciated!