Skip to content

Document Azure Maps rendering in embedded web UI - #2942

Merged
Courtney Wales (Court72) merged 5 commits into
MicrosoftDocs:mainfrom
zook-ms:user/zookkessler/document-iframe-azure-maps
Oct 5, 2026
Merged

Courtney Wales (Court72) merged 5 commits into
MicrosoftDocs:mainfrom
zook-ms:user/zookkessler/document-iframe-azure-maps

Conversation

@zook-ms

@zook-ms zook-ms commented Sep 15, 2026 •

Copy link
Copy Markdown
Contributor

Add instructions for supplying Azure Maps SAS tokens from an embedding host through getMapsToken/postMapsToken messages, including response validation and token lifecycle.

Link to an Azure Maps SAS authentication sample and an Azure Function token-generation example.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 2244aea:

✅ Validation status: passed

File Status Preview URL Details
data-explorer/kusto/api/monaco/host-web-ux-in-iframe.md ✅Succeeded

For more details, please refer to the build report.

@zook-ms

zook-ms commented Sep 15, 2026

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree [company="Microsoft"]

@zook-ms

zook-ms commented Sep 15, 2026

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree company="Microsoft"

@prmerger-automator

Copy link
Copy Markdown
Contributor

zook-ms : 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

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

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 715d93b:

✅ Validation status: passed

File Status Preview URL Details
data-explorer/kusto/api/monaco/host-web-ux-in-iframe.md ✅Succeeded

For more details, please refer to the build report.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

Qualify the Azure Function link to require host-specific authentication and origin checks.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Documents Azure Maps SAS token delivery for embedded web UI rendering.

Changes:

  • Adds getMapsToken/postMapsToken messaging guidance.
  • Documents validation, renewal, and security considerations.
  • Links Azure Maps authentication and Azure Function examples.
File summaries
File Summary
data-explorer/kusto/api/monaco/host-web-ux-in-iframe.md Adds Azure Maps embedding and SAS-token documentation.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread data-explorer/kusto/api/monaco/host-web-ux-in-iframe.md Outdated
@Court72

Copy link
Copy Markdown
Contributor

Sreedhar Pelluru (@spelluru)

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 tracking label for the PR review team label Sep 16, 2026
Updated SAS authentication resources and clarified usage of Azure Function code.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 7ed8045:

✅ Validation status: passed

File Status Preview URL Details
data-explorer/kusto/api/monaco/host-web-ux-in-iframe.md ✅Succeeded

For more details, please refer to the build report.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 35a3f36:

✅ Validation status: passed

File Status Preview URL Details
data-explorer/kusto/api/monaco/host-web-ux-in-iframe.md ✅Succeeded

For more details, please refer to the build report.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 68bcf1c:

✅ Validation status: passed

File Status Preview URL Details
data-explorer/kusto/api/monaco/host-web-ux-in-iframe.md ✅Succeeded

For more details, please refer to the build report.

@zook-ms

zook-ms commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor Author

#sign-off

@zook-ms

zook-ms commented Oct 4, 2026

Copy link
Copy Markdown
Contributor Author

Courtney Wales (@Court72) can we merge this one?

@prmerger-automator

Copy link
Copy Markdown
Contributor

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. Sreedhar Pelluru (@spelluru)

@Court72

Courtney Wales (Court72) commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Hi zook-ms,

In the public repo, pull requests should be signed off by the author, another member of the content team, or a PM.

Sreedhar Pelluru (@spelluru) - Could you take a look? Thanks!

I see that the listed ms.reviewer has approved the changes, so I'll go ahead and merge. Thanks!

@Court72
Courtney Wales (Court72) merged commit 053e5af into MicrosoftDocs:main Oct 5, 2026
3 checks passed
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.

5 participants