Skip to content

fix: Enhance Deployment Script for Tenant ID Prompt and Fix Chatroom UI Reset Bug#401

Merged
Prajwal-Microsoft merged 4 commits intomainfrom
dev
Jun 23, 2025
Merged

fix: Enhance Deployment Script for Tenant ID Prompt and Fix Chatroom UI Reset Bug#401
Prajwal-Microsoft merged 4 commits intomainfrom
dev

Conversation

@Prasanjeet-Microsoft
Copy link
Copy Markdown
Contributor

@Prasanjeet-Microsoft Prasanjeet-Microsoft commented Jun 20, 2025

Purpose

  • Enhancement: The PowerShell deployment script now prompts for and handles the Azure Tenant ID, improving manual deployment flows. It displays the Tenant ID in the summary and ensures accurate authentication.
  • Bug Fix: Addresses a frontend bug where the chatRoom textarea did not reset properly after sending a message. Also improves error logging in the chatRoom component.

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:

In the deployment script:

  • Confirm the script prompts for Tenant ID if not provided.
  • Check that Azure login uses the correct Tenant ID in manual mode.
  • Ensure the Tenant ID is displayed in the deployment summary.

In the frontend chatRoom:

  • Verify the textarea resets after each message.
  • Ensure error logs are informative if an API request fails.

Prasanjeet-Microsoft and others added 2 commits June 19, 2025 21:04
… sending a message

2.Updated PowerShell deployment script to use az login --tenant <tenantId> for multi-tenant support
fix: Token count reset issue in DKM chat UI and enhance deployment script with tenant-specific login
Prasanjeet-Microsoft and others added 2 commits June 20, 2025 16:55
fix: CI pipeline failure resolved by adding AZURE_TENANT_ID input to workflow
@Prajwal-Microsoft Prajwal-Microsoft merged commit 213e928 into main Jun 23, 2025
8 checks passed
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

2 participants