Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: change Event class generator for EventDelta #54

Closed
pietroMonta42 opened this issue Jun 18, 2024 · 1 comment · Fixed by #55
Closed

feat: change Event class generator for EventDelta #54

pietroMonta42 opened this issue Jun 18, 2024 · 1 comment · Fixed by #55
Assignees
Labels
bug Something isn't working e0 🌵 Low effort i1 ⚡️ Medium impact
Milestone

Comments

@pietroMonta42
Copy link

with the new API, now the ThreadMessageDelta is:

class ThreadMessageDelta(BaseModel):
    data: MessageDeltaEvent
    """Represents a message delta i.e.

    any changed fields on a message during streaming.
    """

    event: Literal["thread.message.delta"]
    
 class MessageDeltaEvent(BaseModel):
    id: str
    """The identifier of the message, which can be referenced in API endpoints."""

    delta: MessageDelta
    """The delta containing the fields that have changed on the Message."""

    object: Literal["thread.message.delta"]
    """The object type, which is always `thread.message.delta`."""
@mharrisb1 mharrisb1 self-assigned this Jun 18, 2024
@mharrisb1 mharrisb1 added bug Something isn't working e0 🌵 Low effort i1 ⚡️ Medium impact labels Jun 18, 2024
@mharrisb1 mharrisb1 added this to the v0.7.x milestone Jun 18, 2024
@mharrisb1
Copy link
Owner

Thanks @pietroMonta42 I'll push a fix today or tomorrow whenever I get a second to make the change

@mharrisb1 mharrisb1 linked a pull request Jun 19, 2024 that will close this issue
@mharrisb1 mharrisb1 moved this from Todo to In Progress in openai-responses-python Jun 19, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in openai-responses-python Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working e0 🌵 Low effort i1 ⚡️ Medium impact
Projects
Development

Successfully merging a pull request may close this issue.

2 participants