Skip to content

fix(core): add missing type parameter to chat content constructors - #757

Merged
jrriehl merged 1 commit into
mainfrom
fix/add-missing-type-parameter-to-chat-content-constructors
Aug 5, 2025
Merged

jrriehl merged 1 commit into
mainfrom
fix/add-missing-type-parameter-to-chat-content-constructors

Conversation

@jrriehl

@jrriehl jrriehl commented Aug 5, 2025

Copy link
Copy Markdown
Contributor

Proposed Changes

Fix proposed by @tanaygodse in #756.

Types of changes

What type of change does this pull request make (put an x in the boxes that apply)?

  • Bug fix (non-breaking change that fixes an issue).
  • New feature added (non-breaking change that adds functionality).
  • Breaking change (fix or feature that would cause existing functionality to stop working as expected).
  • Documentation update.
  • Something else (e.g., tests, scripts, example, deployment, infrastructure).

Checklist

Put an x in the boxes that apply:

  • I have read the CONTRIBUTING guide
  • Checks and tests pass locally

If applicable

  • I have added tests that prove my fix is effective or that my feature works
  • I have added/updated the documentation (executed the script in python/scripts/generate_api_docs.py)

@jrriehl
jrriehl marked this pull request as ready for review August 5, 2025 09:09
@jrriehl jrriehl linked an issue Aug 5, 2025 that may be closed by this pull request
@jrriehl
jrriehl merged commit bb90a5b into main Aug 5, 2025
13 checks passed
@jrriehl
jrriehl deleted the fix/add-missing-type-parameter-to-chat-content-constructors branch August 5, 2025 09:17
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.

Error in new constructors with default values for chat protocol

1 participant