Closed Archie-Miller closed 1 month ago
Attention: Patch coverage is 70.00000%
with 12 lines
in your changes missing coverage. Please review.
Project coverage is 54.90%. Comparing base (
41363bd
) to head (9fedb96
). Report is 24 commits behind head on develop/durable.
Files with missing lines | Patch % | Lines |
---|---|---|
...braries/Opc.Ua.Client/Subscription/Subscription.cs | 70.00% | 12 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Proposed changes
Implement a helper method in Opc.Ua.Client.Subscription that calls the Server SetSubscriptionDurable method.
In the ConsoleReferenceClient, Implement a sample implementation that shows the functionality of SetSubscriptionDurable from the client's perspective of pausing a subscription, then creating a new session and subscription and after transfer, retrieve all values with loosing any.
Also in ConsoleReferenceClient, allow for the retrieval of event data, with and without Durable Subscriptions.
Related Issues
https://github.com/orgs/OPCFoundation/projects/1/views/18 Task 1 - Addfunctionto client SDK to support SetSubscriptionDurable
Types of changes
What types of changes does your code introduce? Put an
x
in the boxes that apply. You can also fill these out after creating the PR.Checklist
Put an
x
in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.Further comments