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
Adds support for Unsubscribe Configuration #417
Conversation
Signed-off-by: Bernd Verst <4535280+berndverst@users.noreply.github.com>
Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: GitHub <noreply@github.com>
Codecov Report
@@ Coverage Diff @@
## master #417 +/- ##
==========================================
- Coverage 89.58% 89.40% -0.19%
==========================================
Files 55 55
Lines 2486 2500 +14
==========================================
+ Hits 2227 2235 +8
- Misses 259 265 +6
Continue to review full report at Codecov.
|
Signed-off-by: GitHub <noreply@github.com>
@@ -671,23 +672,32 @@ def __init__(self): | |||
def get_items(self): | |||
return self.items | |||
|
|||
def watch_configuration(self, stub, store_name, keys, config_metadata): | |||
def watch_configuration(self, stub: api_service_v1.DaprStub, store_name: str, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the Java SDK, we have created a new interface to keep Alpha APIs separate. Making it clear that they can break.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. This shows another difference in local decision among SDK maintainers. The alpha APIs are not distinguished from the stable ones.
Description
Adds support for unsubscribe configuration
Issue reference
Fixes #414
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list: