Skip to content

Comments

Adjusting default max conversation id length#196

Merged
axelsrz merged 1 commit intomainfrom
users/robrandao/path-len-adjust
Oct 22, 2025
Merged

Adjusting default max conversation id length#196
axelsrz merged 1 commit intomainfrom
users/robrandao/path-len-adjust

Conversation

@rodrigobr-msft
Copy link
Contributor

This pull request makes a minor configuration update to the ConversationsOperations class. The maximum allowed length for a conversation ID has been reduced from 200 to 150 characters to better align with system constraints.

  • Reduced the default value of _max_conversation_id_length from 200 to 150 in the ConversationsOperations class (connector_client.py).

Copilot AI review requested due to automatic review settings October 22, 2025 20:43
@rodrigobr-msft rodrigobr-msft marked this pull request as ready for review October 22, 2025 20:43
@rodrigobr-msft rodrigobr-msft requested a review from a team as a code owner October 22, 2025 20:43
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR reduces the default maximum conversation ID length from 200 to 150 characters in the ConversationsOperations class to align with system constraints.

Key Changes:

  • Updated the default value of _max_conversation_id_length from 200 to 150 characters

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@axelsrz axelsrz merged commit 1e4f35f into main Oct 22, 2025
10 checks passed
@axelsrz axelsrz deleted the users/robrandao/path-len-adjust branch October 22, 2025 20:52
Copilot AI mentioned this pull request Dec 2, 2025
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