Skip to content

Update image upload guidelines in documentation#720

Open
aavyas77 wants to merge 2 commits into
MicrosoftDocs:mainfrom
aavyas77:patch-2
Open

Update image upload guidelines in documentation#720
aavyas77 wants to merge 2 commits into
MicrosoftDocs:mainfrom
aavyas77:patch-2

Conversation

@aavyas77
Copy link
Copy Markdown
Contributor

@aavyas77 aavyas77 commented Apr 10, 2026 •
edited by v-shils
Loading

PR review feedback: #720 (comment)

Clarified image URL requirements and limitations for Vision service.

@aavyas77
Update image upload guidelines in documentation
90b61ba 
Clarified image URL requirements and limitations for Vision service.
@prmerger-automator
Copy link
Copy Markdown
Contributor

prmerger-automator Bot commented Apr 10, 2026

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

@prmerger-automator
Copy link
Copy Markdown
Contributor

prmerger-automator Bot commented Apr 10, 2026

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

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 10, 2026

Learn Build status updates of commit 90b61ba:

✅ Validation status: passed

File Status Preview URL Details
articles/foundry/openai/includes/how-to-gpt-with-vision-content.md ✅Succeeded

For more details, please refer to the build report.

@v-regandowner
Copy link
Copy Markdown
Contributor

v-regandowner commented Apr 10, 2026

@PatrickFarley

Can you review the proposed changes?

IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

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

@prmerger-automator prmerger-automator Bot added the aq-pr-triaged C+L Pull Request Review Team label label Apr 10, 2026
@v-regandowner v-regandowner requested a review from Copilot April 10, 2026 15:29
Copilot AI reviewed Apr 10, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Vision-enabled chat documentation include to clarify requirements and limitations around supplying image URLs to the Vision service.

Changes:

  • Adds guidance that image URLs must be publicly accessible.
  • Adds explanation tying lack of private endpoint/VNet access to how the service fetches images.

Comment thread articles/foundry/openai/includes/how-to-gpt-with-vision-content.md Outdated
Comment thread articles/foundry/openai/includes/how-to-gpt-with-vision-content.md Outdated
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 27, 2026

This pull request has been inactive for at least 14 days. If you are finished with your changes, don't forget to sign off. See the contributor guide for instructions.
Get Help
Docs Support Teams Channel
Resolve Merge Conflict

@github-actions github-actions Bot added the inactive This PR is inactive for more than 14 days label Apr 27, 2026
@aavyas77
Copy link
Copy Markdown
Contributor Author

aavyas77 commented May 26, 2026

@PatrickFarley any updates on this?

@github-actions github-actions Bot removed the inactive This PR is inactive for more than 14 days label May 26, 2026
@prmerger-automator
Copy link
Copy Markdown
Contributor

prmerger-automator Bot commented May 26, 2026

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

@v-dirichards
Copy link
Copy Markdown
Contributor

v-dirichards commented May 26, 2026

@PatrickFarley

Can you review this old PR and determine whether it needs to be closed or merged?

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

@PatrickFarley
Copy link
Copy Markdown
Contributor

PatrickFarley commented May 28, 2026

#sign-off

@v-regandowner
Copy link
Copy Markdown
Contributor

v-regandowner commented May 28, 2026

Due to the upcoming conference, all merging and publishing is on hold until the morning of June 2nd.

This PR is ready for merge on June 2nd.

#label:"aq-pr-triaged"
#label: "event-triaged-to-10AM"
#label: "missed-content-complete"
#label: "ready-to-merge"

@prmerger-automator prmerger-automator Bot added event-triaged-to-10AM missed-content-complete Label used during release conference to identify PRs in content freeze labels May 28, 2026
PatrickFarley
PatrickFarley reviewed Jun 1, 2026
View reviewed changes
Comment thread articles/foundry/openai/includes/how-to-gpt-with-vision-content.md Outdated
@v-shils v-shils removed missed-content-complete Label used during release conference to identify PRs in content freeze event-triaged-to-10AM labels Jun 2, 2026
@v-dirichards
Copy link
Copy Markdown
Contributor

v-dirichards commented Jun 2, 2026

#hold-off

@aavyas77 @PatrickFarley
Update articles/foundry/openai/includes/how-to-gpt-with-vision-conten…
9b98a81 
…t.md

Co-authored-by: Patrick Farley <pafarley@microsoft.com>
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Jun 3, 2026

Learn Build status updates of commit 9b98a81:

✅ Validation status: passed

File Status Preview URL Details
articles/foundry/openai/includes/how-to-gpt-with-vision-content.md ✅Succeeded

For more details, please refer to the build report.

@v-dirichards
Copy link
Copy Markdown
Contributor

v-dirichards commented Jun 3, 2026

@PatrickFarley Could you review this proposed update to your article and enter #sign-off in a comment if it's ready to merge?

Thanks!

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

Reviewers

@v-dirichards v-dirichards v-dirichards left review comments

Copilot code review Copilot Copilot left review comments

+1 more reviewer

@PatrickFarley PatrickFarley PatrickFarley left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@PatrickFarley PatrickFarley

Labels

aq-pr-triaged C+L Pull Request Review Team label azure-ai-foundry/svc Change sent to author do-not-merge

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@aavyas77 @v-regandowner @v-dirichards @PatrickFarley @v-shils