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

Update user-convertexternaltointernalmemberuser.md #9501

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

Conversation

sirlupinwatson
Copy link

There is a small error in the function parameter from entra.microsoft.com where you can add additional feature to your API call, I did not find where it was so I have included it as a comment in the README.md

Important

Required for API changes:

  • Link to API.md file: ADD LINK HERE
  • Link to PR for public-facing schema changes (schema-Prod-beta/v1.0.csdl): Link to the .md

Notes:

I did not find the actual error but I can include a screenshot

I use the entra ID admin center to update parameter or see a list of available feature and connect with this link, I removed the add ID...
site link


Screenshot_25-1-2025_4659_entra microsoft com

Important

The following guidance is for Microsoft employees only. Community contributors can ignore this message; our content team will manage the status.

After you've created your PR, expand this section for tips and additional instructions.
  • do not merge is the default PR status and is automatically added to all open PRs that don't have the ready to merge label.
  • Add the ready for content review label to start a review. Only PRs that have met the minimum requirements for content review and have this label are reviewed.
  • If your content reviewer requests changes, review the feedback and address accordingly as soon as possible to keep your pull request moving forward. After you address the feedback, remove the changes requested label, add the review feedback addressed label, and select the Re-request review icon next to the content reviewer's alias. If you can't add labels, add a comment with #feedback-addressed to the pull request.
  • After the content review is complete, your reviewer will add the content review complete label. When the updates in this PR are ready for external customers to use, replace the do not merge label with ready to merge and the PR will be merged within 24 working hours.
  • Pull requests that are inactive for more than 6 weeks will be automatically closed. Before that, you receive reminders at 2 weeks, 4 weeks, and 6 weeks. If you still need the PR, you can reopen or recreate the request.

For more information, see the Content review process summary.

There is a small error in the function parameter from entra.microsoft.com where you can add additional feature to your API call, I did not find where it was so I have included it as a comment in the README.md
Copy link

Learn Build status updates of commit 20c9e6a:

✅ Validation status: passed

File Status Preview URL Details
api-reference/beta/api/user-convertexternaltointernalmemberuser.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@@ -9,7 +9,10 @@ doc_type: apiPageType
ms.date: 10/15/2024
---

# user: convertExternalToInternalMemberUser
# user: convertExternalToInternalMemberUser
<!-- I am not sure where is the typo error so I will include a comment here since it's from the function here as memeber: -->
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not clear about the issue you're trying to address. Also, there's a typo in "member". @sirlupinwatson

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi there hope you are doing well! I was trying to figure out how to, or where to update the Interface in itself and see if this could have been useful in some way. Thank you for the mention I appreciate it!

1- Go to Entra Admin Dashboard
2- In the Registered App you head over the management and look for the API permissions.
3- Search for Microsoft then select Microsoft Graph
4- Select Delegated Permissions and scroll down to User-ConvertToInternal

part1
part2

review feedback addressed

Copy link
Contributor

@Danielabom Danielabom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sirlupinwatson, please see my comment.

@Danielabom Danielabom self-assigned this Feb 3, 2025
@Danielabom Danielabom added content review in progress do not merge Use this label to indicate to the repository admins that your PR Is not ready to merge into master. changes requested labels Feb 3, 2025
Copy link
Contributor

@sirlupinwatson, your content reviewer has requested changes to your pull request. Please review feedback and address accordingly as soon as possible to keep your pull request moving forward.

Once you have addressed the feedback, please add the review feedback addressed label and select the Re-request review icon next to the content reviewer's alias.

If you cannot add labels, please add a comment with #feedback-addressed to the pull request.

Check the content review workflow here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes requested content review in progress do not merge Use this label to indicate to the repository admins that your PR Is not ready to merge into master.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants