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

🐛 Bug Report: GetStream missing / or lacks properties in POST requests #5706

Closed
2 tasks done
lillheaton opened this issue Jun 11, 2024 · 1 comment
Closed
2 tasks done
Labels

Comments

@lillheaton
Copy link

📜 Description

getStream requires other or potentially more fields in order to make request to make sense?

https://getstream.io/chat/docs/rest/#product:chat-sendmessagehttps://getstream.io/chat/docs/rest/#product:chat-sendmessage

My assumption is that the above API path is the common usage for getStream.

It requires body property "message" that contains multiple properties.
Currently the Novu provider only sends the data "text"

https://github.com/novuhq/novu/blob/next/packages/providers/src/lib/getstream/getstream.provider.ts#L24

👟 Reproduction steps

N/A

👍 Expected behavior

Allow for more or other property fields to be sent to getStream

👎 Actual Behavior with Screenshots

N/A

Novu version

Novu SaaS

npm version

No response

node version

No response

📃 Provide any additional context for the Bug.

No response

👀 Have you spent some time to check if this bug has been raised before?

  • I checked and didn't find a similar issue

🏢 Have you read the Contributing Guidelines?

Are you willing to submit PR?

None

Copy link

linear bot commented Jun 11, 2024

@linear linear bot closed this as not planned Won't fix, can't repro, duplicate, stale Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant