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

Google Pub/Sub gRPC: docs for subscription in AcknowledgeRequest #2677

Merged
merged 2 commits into from
May 25, 2021
Merged

Google Pub/Sub gRPC: docs for subscription in AcknowledgeRequest #2677

merged 2 commits into from
May 25, 2021

Conversation

derekchan
Copy link
Contributor

References #2354

This adds, in the documentation examples, .withSubscription to the AcknowledgeRequest construction. The current example code would have acks failing with an invalid resource error.

@lightbend-cla-validator

Hi @derekchan,

Thank you for your contribution! We really value the time you've taken to put this together.

Before we proceed with reviewing this pull request, please sign the Lightbend Contributors License Agreement:

https://www.lightbend.com/contribute/cla

@ennru
Copy link
Member

ennru commented May 24, 2021

Thank you for suggesting this fix!

The code isn't formatted with scalafmtAll https://travis-ci.com/github/akka/alpakka/jobs/507463611#L780

Copy link
Member

@ennru ennru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@ennru ennru changed the title Google Pub/Sub gRPC: Updating docs to set subscription in AcknowledgeRequest Google Pub/Sub gRPC: docs to set subscription in AcknowledgeRequest May 25, 2021
@ennru ennru changed the title Google Pub/Sub gRPC: docs to set subscription in AcknowledgeRequest Google Pub/Sub gRPC: docs for subscription in AcknowledgeRequest May 25, 2021
@ennru ennru merged commit 02b037c into akka:master May 25, 2021
gkatzioura pushed a commit to gkatzioura/alpakka that referenced this pull request Jun 3, 2021
gkatzioura pushed a commit to gkatzioura/alpakka that referenced this pull request Jun 3, 2021
gkatzioura pushed a commit to gkatzioura/alpakka that referenced this pull request Jun 3, 2021
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.

3 participants