Skip to content

Conversation

@CahidArda
Copy link
Collaborator

No description provided.

@linear
Copy link

linear bot commented Apr 16, 2025

@CahidArda CahidArda force-pushed the DX-1821-deprecate-llms branch from 5fc5473 to 493b571 Compare April 16, 2025 10:27
@CahidArda CahidArda requested a review from mdumandag April 16, 2025 12:44
Copy link
Collaborator

@mdumandag mdumandag left a comment

Choose a reason for hiding this comment

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

We also need to remove/deprecate api fields (like in dlq filter/event filter/messages/destinations/... etc).

@yunusemreozdemir
Copy link
Contributor

I think there is a model-provider mismatch here, same for async test:
https://github.com/upstash/qstash-py/blob/DX-1821-deprecate-llms/tests/test_message.py#L241

Copy link
Collaborator

@mdumandag mdumandag left a comment

Choose a reason for hiding this comment

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

there is one more leftover api field that should be removed:
see message.py#298

README.md Outdated
```python
from qstash import QStash
from qstash.chat import upstash
from qstash.chat import openai
Copy link
Collaborator

Choose a reason for hiding this comment

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

now, there is a duplicate example. see the line 103, Publish a JSON message to LLM Using Custom Providers

we should remove this or that

@mdumandag
Copy link
Collaborator

Apart from this PR, I think we have renamed events to logs in JS SDK, we can make the same here and remove the events api altogether since this is a major version bump

@CahidArda CahidArda merged commit 5c64d9b into main Apr 28, 2025
@CahidArda CahidArda deleted the DX-1821-deprecate-llms branch April 28, 2025 09:57
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.

4 participants