Skip to content

fix: Token count reset issue in DKM chat UI and enhance deployment script with tenant-specific login#400

Merged
Prajwal-Microsoft merged 1 commit intodevfrom
PSL-BUG-19531
Jun 19, 2025
Merged

fix: Token count reset issue in DKM chat UI and enhance deployment script with tenant-specific login#400
Prajwal-Microsoft merged 1 commit intodevfrom
PSL-BUG-19531

Conversation

@Prasanjeet-Microsoft
Copy link
Copy Markdown
Contributor

Purpose

This PR addresses a UI issue in the DKM chat interface and improves the deployment script to support tenant-specific authentication:

  • UI Fix: Token/Character count now resets correctly after each message is sent and a response is received.
  • Deployment Update: PowerShell script modified to allow login using a specified tenant ID, improving compatibility in multi-tenant environments

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:

  • Token count resets to 0/500 after sending a message and receiving a response
  • Chat functionality works as expected with no UI regressions
  • PowerShell deployment runs successfully with tenant-specific login
  • No impact on users with default single-tenant login

… sending a message

2.Updated PowerShell deployment script to use az login --tenant <tenantId> for multi-tenant support
@Prajwal-Microsoft Prajwal-Microsoft merged commit 7da6c1b into dev Jun 19, 2025
5 checks passed
@Prasanjeet-Microsoft Prasanjeet-Microsoft deleted the PSL-BUG-19531 branch July 3, 2025 19:45
@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