Skip to content

Conversation

lhoupert
Copy link

@lhoupert lhoupert commented Feb 11, 2025

Just raising a small change to the doc after facing some problems when trying to upload and download promptbooks from uksouth region

Copy link
Contributor

@Copilot 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.

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment on lines +90 to +91
- If after running the upload promptbook script the new uploaded promptbooks are not visible on the webportal, make sure to the region URI is set up properly in the powershell script `https://api.securitycopilot.microsoft.com/geo/<region>/promptbooks` (4 regions are currently supported East US, West Europe, UK South, and Australia East)

Copy link
Preview

Copilot AI Feb 18, 2025

Choose a reason for hiding this comment

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

The phrase 'make sure to the region URI is set up properly' should be corrected to 'make sure the region URI is set up properly'.

Suggested change
- If after running the upload promptbook script the new uploaded promptbooks are not visible on the webportal, make sure to the region URI is set up properly in the powershell script `https://api.securitycopilot.microsoft.com/geo/<region>/promptbooks` (4 regions are currently supported East US, West Europe, UK South, and Australia East)
- If after running the upload promptbook script the new uploaded promptbooks are not visible on the webportal, make sure the region URI is set up properly in the powershell script `https://api.securitycopilot.microsoft.com/geo/<region>/promptbooks` (4 regions are currently supported East US, West Europe, UK South, and Australia East)

Copilot uses AI. Check for mistakes.

@KwachSean KwachSean requested a review from Copilot March 3, 2025 07:35
Copy link
Contributor

@Copilot 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.

PR Overview

This PR updates the documentation to address troubleshooting steps when promptbooks uploaded from the UK South region are not visible on the web portal.

  • Added a troubleshooting case to clarify configuration of the region URI in the PowerShell script.

Reviewed Changes

File Description
Promptbook samples/Powershell to Manage Promptbooks/readme.md Added a note about ensuring the correct region URI is set in the script

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment on lines +90 to +91
- If after running the upload promptbook script the new uploaded promptbooks are not visible on the webportal, make sure to the region URI is set up properly in the powershell script `https://api.securitycopilot.microsoft.com/geo/<region>/promptbooks` (4 regions are currently supported East US, West Europe, UK South, and Australia East)

Copy link
Preview

Copilot AI Mar 3, 2025

Choose a reason for hiding this comment

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

There is a grammatical issue in the added line. Consider removing the extra 'to' so that it reads 'make sure the region URI is set up properly' for clarity.

Suggested change
- If after running the upload promptbook script the new uploaded promptbooks are not visible on the webportal, make sure to the region URI is set up properly in the powershell script `https://api.securitycopilot.microsoft.com/geo/<region>/promptbooks` (4 regions are currently supported East US, West Europe, UK South, and Australia East)
- If after running the upload promptbook script the new uploaded promptbooks are not visible on the webportal, make sure the region URI is set up properly in the powershell script `https://api.securitycopilot.microsoft.com/geo/<region>/promptbooks` (4 regions are currently supported East US, West Europe, UK South, and Australia East)

Copilot uses AI. Check for mistakes.

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

Successfully merging this pull request may close these issues.

1 participant