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

Update streaming subscription to understand new initial response #601

Merged

yaron2 merged 4 commits into dapr:main from JoshVanL:streaming-sub-initial-response

Jul 23, 2024

Merged

Update streaming subscription to understand new initial response #601

Update streamsub name in validate examples

Codecov / codecov/patch failed Jul 23, 2024 in 0s

0.00% of diff hit (target 58.04%)

View this Pull Request on Codecov

0.00% of diff hit (target 58.04%)

Annotations

Check warning on line 57 in client/subscribe.go

codecov / codecov/patch

client/subscribe.go#L57

Added line #L57 was not covered by tests

Check warning on line 61 in client/subscribe.go

codecov / codecov/patch

client/subscribe.go#L61

Added line #L61 was not covered by tests

Check warning on line 104 in client/subscribe.go

codecov / codecov/patch

client/subscribe.go#L104

Added line #L104 was not covered by tests

Check warning on line 108 in client/subscribe.go

codecov / codecov/patch

client/subscribe.go#L108

Added line #L108 was not covered by tests

Check warning on line 188 in client/subscribe.go

codecov / codecov/patch

client/subscribe.go#L187-L188

Added lines #L187 - L188 were not covered by tests

Check warning on line 212 in client/subscribe.go

codecov / codecov/patch

client/subscribe.go#L212

Added line #L212 was not covered by tests

Check warning on line 224 in client/subscribe.go

codecov / codecov/patch

client/subscribe.go#L222-L224

Added lines #L222 - L224 were not covered by tests

Check warning on line 230 in client/subscribe.go

codecov / codecov/patch

client/subscribe.go#L227-L230

Added lines #L227 - L230 were not covered by tests

View more details on Codecov