Skip to content

Conversation

@lilyydu
Copy link
Collaborator

@lilyydu lilyydu commented Nov 13, 2025

added typescript and python as .NET implementation will be delayed with other refactorings

@lilyydu lilyydu marked this pull request as ready for review November 21, 2025 22:14

To properly configure your regional bot, you will need to make a few updates. This example uses West Europe, but follow the equivalent for other regions.

1. In `azurebot.bicep`, replace all `global` occurrences to `westeurope`
Copy link
Collaborator

Choose a reason for hiding this comment

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

This also only applies when the bot is being created.
We should call out that this is only if they use ATK. It would be also good to call the equivalent guidance if they configure it manually.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

added a remark that this is for ATK, by manual you mean adding the steps for Azure?

Copy link
Collaborator

Choose a reason for hiding this comment

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

yeah, because if they don't have an azure bicep file, this doesn't apply

app = App(
default_connection_name='graph',
api_client_settings=ApiClientSettings(
oauth_url=os.environ.get("OAUTH_URL") # Value would be "https://europe.token.botframework.com"
Copy link
Collaborator

@heyitsaamir heyitsaamir Nov 26, 2025

Choose a reason for hiding this comment

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

You don't need this at all right? It should be updates in the .env file or wherever they set their environment variables

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think it's still good to showcase here so they know what the value should look like

Copy link
Collaborator

@heyitsaamir heyitsaamir Nov 27, 2025

Choose a reason for hiding this comment

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

Hm, this actually makes it look like this is a required thing they need to provide in code. And it's not clear that it'll automatically use this config from the environment variable.

You could show the same thing but instead of here, show it in an .env file instead

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.

3 participants