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.

Sign up for GitHub

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

Jump to bottom

Extend ChatUIProvider to have stream #1054

Closed
HarshRajat opened this issue Jan 23, 2024 — with Slack · 0 comments · Fixed by #1058
Closed

Extend ChatUIProvider to have stream #1054

HarshRajat opened this issue Jan 23, 2024 — with Slack · 0 comments · Fixed by #1058
Assignees
@mishramonalisha76
Labels
frontend Frontend related

Comments

@HarshRajat Slack
Copy link
Contributor

Rationale: many dApps including our dApp uses stream for other things as well, if ChatUIProvider has pushuser but not stream then stream should be created for other components but if user and stream is passed then that should be taken.

Also pushUser should be simply user
@HarshRajat HarshRajat added the frontend Frontend related label Jan 23, 2024 — with Slack
@mishramonalisha76 mishramonalisha76 linked a pull request Jan 23, 2024 that will close this issue
fix: changed pushUser to user #1058
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mishramonalisha76 mishramonalisha76

Labels
frontend Frontend related
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: changed pushUser to user push-protocol/push-sdk
3 participants
@HarshRajat @corlard3y @mishramonalisha76