Skip to content

Update driveitem-get-content-format.md - #9564

Open
Westley (westleyMS) wants to merge 2 commits into
microsoftgraph:mainfrom
westleyMS:patch-3
Open

Update driveitem-get-content-format.md#9564
Westley (westleyMS) wants to merge 2 commits into
microsoftgraph:mainfrom
westleyMS:patch-3

Conversation

@westleyMS

Copy link
Copy Markdown
Contributor

need to add a note on file size limitation per the PG (gaddanki@microsoft.com)

Important

Required for API changes:

  • Link to API.md file: - reference/v1.0/api/driveitem-get-content-format.md

The limit for conversion of files according to gaddanki@microsoft.com is 2 MB. Going over this results often in timeouts although sometimes it works.

need to add a note on file size limitation per the PG (gaddanki@microsoft.com)
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 0a9aa9c:

✅ Validation status: passed

File Status Preview URL Details
api-reference/v1.0/api/driveitem-get-content-format.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Comment thread api-reference/v1.0/api/driveitem-get-content-format.md Outdated
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 3d64b30:

✅ Validation status: passed

File Status Preview URL Details
api-reference/v1.0/api/driveitem-get-content-format.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@Danielabom

Daniela Bonilla Montero (Danielabom) commented May 23, 2025

Copy link
Copy Markdown
Contributor

Westley (@westleyMS), I noticed it was named incorrectly. This is the format we follow for GET methods.

{CFC951FB-E6A9-47DB-8657-B22D9D3446F2}

Also, we're not following the SEO requirements to make sure your file is discoverable. I can fix that. But, can you help me choose the correct name for the file? These are our guidelines (https://eng.ms/docs/products/microsoft-graph-service/microsoft-graph/document-apis/file-naming).

@github-actions

github-actions Bot commented Jul 5, 2025

Copy link
Copy Markdown

This pull request has conflicting changes, the author must resolve the conflicts before this pull request can be merged.

@v-regandowner

Copy link
Copy Markdown

Westley (@westleyMS) - Can you respond to the comment above?

@MicrosoftDocs/public-repo-pr-review-team

@v-regandowner Regan Downer (v-regandowner) added the aq-pr-triaged PR has been processed by the Aquent PR review team label Jul 24, 2026
@westleyMS

Copy link
Copy Markdown
Contributor Author

Regan Downer (@v-regandowner) Daniela Bonilla Montero (@Danielabom) I am not familiar with any of the naming conventions nor did I create or name the file.
The operation is parentresourcename-get-navigationpropertyname-method, it looks correct to me.
The public page is here-
https://learn.microsoft.com/en-us/graph/api/driveitem-get-content-format?view=graph-rest-1.0&tabs=http

The call is-
GET /drive/items/{item-id}/content?format={format}

since format is on a query string at the end, it has been included in the name. Keep the name and just add the limitation on size.

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

Labels

aq-pr-triaged PR has been processed by the Aquent PR review team conflicting content review in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants