fix: Dev v4 to main merge#970
Merged
Merged
Conversation
Updated AddPlatform.png and Web.png with latest Azure portal UI. Updated step 6 from 'Add a platform' to 'Add redirect URI' in create_new_app_registration.md. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
docs: Image update
…unctionality; adjust PanelFooter and PanelLeft styles
…lay logic; enhance fetchCurrentUser to handle user info retrieval and error management
docs: Readme and deployment doc update
Dhruvkumar-Microsoft
requested review from
Avijit-Microsoft,
Fr4nc3,
Prajwal-Microsoft,
Roopan-Microsoft,
Vinay-Microsoft,
aniaroramsft,
dgp10801,
marktayl1,
nchandhi and
toherman-msft
as code owners
Dhruvkumar-Microsoft
temporarily deployed
to
production
GitHub Actions
Inactive
Contributor
There was a problem hiding this comment.
Pull request overview
This PR aligns deployment documentation and azd parameterization for the v4 branch by updating region guidance, correcting/standardizing environment parameter naming, and fixing minor documentation inaccuracies.
Changes:
- Update README recommended Azure regions for required services.
- Set a default container image tag (
latest_v4) viaAZURE_ENV_IMAGE_TAGininfra/main.parameters.jsonand document the parameter consistently. - Minor doc corrections (environment name length guidance, UI instruction wording, typo fix).
Reviewed changes
Copilot reviewed 5 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| README.md | Updates example Azure regions to reflect current availability guidance. |
| infra/main.parameters.json | Uses AZURE_ENV_IMAGE_TAG with a latest_v4 default for backend/frontend/MCP image tags. |
| docs/CustomizingAzdParameters.md | Updates env name length guidance and standardizes the image tag env var name/default. |
| docs/create_new_app_registration.md | Adjusts an Azure Portal UI instruction related to redirect URI configuration. |
| docs/azure_app_service_auth_setup.md | Fixes a typo (“idenity” → “identity”) in the auth setup steps. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
chore: Bicep Parameter validation email format change
Roopan-Microsoft
temporarily deployed
to
production
GitHub Actions
Inactive
Roopan-Microsoft
temporarily deployed
to
production
GitHub Actions
Inactive
feat: implementation of logout functionality
Roopan-Microsoft
temporarily deployed
to
production
GitHub Actions
Inactive
Roopan-Microsoft
temporarily deployed
to
production
GitHub Actions
Inactive
![github-code-quality[bot]](https://avatars.githubusercontent.com/u/9919?s=60&v=4){kind=small}
…ng logic - Update email body handling in Bicep validation workflow to include fallback messages when HTML reports are not generated. - Refactor `fetchCurrentUser` to `hydrateCurrentUser` in app slice, improving user state management. - Remove unused `fetchCurrentUser` async thunk and related error handling. - Adjust LoginButton component to streamline authentication logic. - Minor documentation update for clarity in app registration instructions.
Roopan-Microsoft
previously approved these changes
May 11, 2026
fix: copilot comments resolve code changes
Roopan-Microsoft
temporarily deployed
to
production
GitHub Actions
Inactive
Roopan-Microsoft
temporarily deployed
to
production
GitHub Actions
Inactive
Roopan-Microsoft
approved these changes
May 12, 2026
Roopan-Microsoft
approved these changes
May 12, 2026
Prajwal-Microsoft
approved these changes
May 12, 2026
|
🎉 This PR is included in version 4.2.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
This pull request makes several documentation and configuration improvements to ensure consistency and clarity when deploying the solution accelerator. The most important changes include correcting environment parameter names and defaults, updating Azure region recommendations, and clarifying instructions in the documentation.
Parameter and Configuration Updates:
AZURE_ENV_IMAGETAGtoAZURE_ENV_IMAGE_TAG, updated its default value fromlatest_v3tolatest_v4, and ensured consistent usage in both documentation and infrastructure configuration (docs/CustomizingAzdParameters.md,infra/main.parameters.json). [1] [2]Documentation Improvements:
README.mdto reflect currently available regions for required services.docs/CustomizingAzdParameters.md).Minor Corrections:
docs/azure_app_service_auth_setup.md).docs/create_new_app_registration.md).Does this introduce a breaking change?
How to Test
What to Check
Verify that the following are valid
Other Information