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

Removed advice on adding users by GitHub username #14196

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

felpsey
Copy link

@felpsey felpsey commented Jan 9, 2025

Under the "Add users to your organization" heading of "add-organization-users.md", the documentation advises that you may identify users by GitHub username.

I have been unable to add users by GitHub username within my Azure DevOps Organisation either by GitHub username or GitHub email address.

See the screenshots below:

image
image

Additionally, under the "FAQs" heading of "connect-to-github.md" it referes to a "policy for inviting GitHub users". This policy does not exist in my ADO Organisation.

image

Can someone clarify if I am missing additional context or if this feature has been removed from ADO please? I have tentatively removed the advice from add-organization-users.md.

Copy link
Contributor

@felpsey : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@felpsey
Copy link
Author

felpsey commented Jan 9, 2025

After further testing, the 'Invite GitHub users' policy becomes visible only when a Microsoft Entra directory is not connected. However, I am still unable to add a user by GitHub username.

@Jak-MS
Copy link
Contributor

Jak-MS commented Jan 9, 2025

@felpsey - when you're ready for this PR to be reviewed and merged, can you take it out of 'draft' mode and notify @chcomley in a comment? thanks.

@chcomley - please take a look and sign off when you approve. thanks.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@chcomley
Copy link
Member

chcomley commented Jan 9, 2025

@felpsey, I'm currently looking into this, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants