-
Notifications
You must be signed in to change notification settings - Fork 726
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
Add subscribe.v1alpha1 to the API allow list docs #4297
Add subscribe.v1alpha1 to the API allow list docs #4297
Conversation
Signed-off-by: Joni Collinge <[email protected]>
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.
@jjcollinge on line 117, could you try changing the link from:
https://github.com/dapr/dapr/blob/master/pkg/grpc/endpoints.go
to:
That might be holding up the PR build job.
Signed-off-by: Joni Collinge <[email protected]>
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
@holopin-bot @jjcollinge Thanks Joni! 😁 |
Congratulations @jjcollinge, the maintainer of this repository has issued you a badge! Here it is: https://holopin.io/claim/clzv9dyqs34840cidljrj6ff4 This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account. |
Update the API allow list docs to include the
susbcribe.v1alpha1
gRPC API.The DCO check is failing as the base was originally wrong on the PR - the 1 commit in this PR does have the signature.