Skip to content

docs: added misssing parameter and removed unused. #589

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jul 4, 2025
Merged

docs: added misssing parameter and removed unused. #589

merged 1 commit into from
Jul 4, 2025

Conversation

@VishalS-Microsoft
Copy link
Contributor

@VishalS-Microsoft VishalS-Microsoft commented Jul 4, 2025

Purpose

This pull request updates documentation files to improve clarity and provide more accurate parameter descriptions. The most notable changes involve updates to Azure OpenAI location settings and AI Foundry project resource parameters.

Documentation updates:

  • docs/CustomizingAzdParameters.md: Removed the AZURE_ENV_OPENAI_LOCATION parameter and replaced RESOURCE_GROUP_NAME_FOUNDRY with AZURE_EXISTING_AI_PROJECT_RESOURCE_ID to clarify resource reuse for AI Foundry projects.
  • docs/DeploymentGuide.md: Removed the Azure OpenAI Location setting from the deployment guide to align with updated parameter descriptions.

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • ...

Other Information

@Roopan-Microsoft Roopan-Microsoft merged commit 7e0178f into dev Jul 4, 2025
5 checks passed
@Roopan-Microsoft Roopan-Microsoft deleted the psl-missingparameter-clientadvisor branch July 4, 2025 13:24
Roopan-Microsoft pushed a commit that referenced this pull request Jul 9, 2025
@Dhruvkumar-Microsoft @VishalS-Microsoft
fix: Changed the prompt to accurately answer scheduling or time based…
4c4ebf0 
… meeting questions and Readme changes (#593)

* added misssing parameter and removed unused. (#589)

* Changed the prompt to accurately answer scheduling or time based meeting questions (#590)

---------

Co-authored-by: VishalS-Microsoft <v-vishshinde@microsoft.com>
Roopan-Microsoft pushed a commit that referenced this pull request Jul 11, 2025
@VishalS-Microsoft @Dhruvkumar-Microsoft
chore: Azd version changes Dev to Main (#596)
22125fc 
* added misssing parameter and removed unused. (#589)

* Changed the prompt to accurately answer scheduling or time based meeting questions (#590)

* updated docs and azure.yml file (#595)

---------

Co-authored-by: Dhruvkumar-Microsoft <v-dbabariya@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Roopan-Microsoft Roopan-Microsoft Roopan-Microsoft approved these changes

@Avijit-Microsoft Avijit-Microsoft Awaiting requested review from Avijit-Microsoft Avijit-Microsoft is a code owner

@Prajwal-Microsoft Prajwal-Microsoft Awaiting requested review from Prajwal-Microsoft Prajwal-Microsoft is a code owner

@Vinay-Microsoft Vinay-Microsoft Awaiting requested review from Vinay-Microsoft Vinay-Microsoft is a code owner

@aniaroramsft aniaroramsft Awaiting requested review from aniaroramsft aniaroramsft is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@VishalS-Microsoft @Roopan-Microsoft