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

Feature/manual ack #403

Merged
merged 26 commits into from
Apr 18, 2020
Merged

Feature/manual ack #403

merged 26 commits into from
Apr 18, 2020

Conversation

SgtSilvio
Copy link
Member

Motivation
Resolves #304

IMPORTANT: base branch needs to be changed to develop once #398 is merged.

Changes

  • Added manualAcknowledgement parameter to Mqtt3/5Rx/Async/Blocking/ReactorClient.publishes
  • Added manualAcknowledgement parameter to Mqtt3/5RxClient.subscribePublishes(With) and Mqtt3/5AsyncClient.subscribe(With)
  • Deprecated Mqtt3/5RxClient.subscribeStream(With), replaced with subscribePublishes(With)
  • Added Mqtt3/5Publish.acknowledge

@cla-bot cla-bot bot added the cla-signed label Apr 15, 2020
@SgtSilvio SgtSilvio self-assigned this Apr 15, 2020
@SgtSilvio SgtSilvio linked an issue Apr 15, 2020 that may be closed by this pull request
@SgtSilvio SgtSilvio requested a review from sauroter April 16, 2020 07:10
@sauroter
Copy link
Member

I think this works as intended.

@SgtSilvio SgtSilvio force-pushed the feature/automatic-resubscribe branch from 7b7a86e to 828edad Compare April 17, 2020 09:21
@SgtSilvio SgtSilvio force-pushed the feature/manual-ack branch from 8a5e4fe to afc6d73 Compare April 17, 2020 09:21
@sauroter sauroter closed this Apr 17, 2020
@sauroter sauroter reopened this Apr 17, 2020
@SgtSilvio SgtSilvio mentioned this pull request Apr 17, 2020
@SgtSilvio SgtSilvio force-pushed the feature/manual-ack branch 2 times, most recently from f8c3a9e to 4f8bfd5 Compare April 18, 2020 20:10
@SgtSilvio SgtSilvio changed the base branch from feature/automatic-resubscribe to develop April 18, 2020 20:43
@SgtSilvio SgtSilvio force-pushed the feature/manual-ack branch from 4f8bfd5 to c2df96c Compare April 18, 2020 20:46
@SgtSilvio SgtSilvio merged commit 5bb2039 into develop Apr 18, 2020
@SgtSilvio SgtSilvio deleted the feature/manual-ack branch April 18, 2020 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Manual acknowledgement
2 participants