Skip to content

Conversation

@bledeamarius
Copy link

Update docs to include the subscrition name for python v1. My containerized azure function was not starting due to this missing parameter.

Update docs to include the subscrition name for python v1
@prmerger-automator
Copy link
Contributor

@bledeamarius : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 4d35b98:

✅ Validation status: passed

File Status Preview URL Details
articles/azure-functions/functions-bindings-service-bus-trigger.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@Court72
Copy link
Contributor

Court72 commented Apr 11, 2024

@ggailey777

Can you review the proposed changes?

When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label Apr 11, 2024
@bledeamarius
Copy link
Author

Hi @Court72. Is it possible to add another reviewer? Thanks

@Court72
Copy link
Contributor

Court72 commented Apr 16, 2024

Hi @bledeamarius,

Anyone can review the PR, but PRMerger will accept sign-off from the listed author only.

@bledeamarius
Copy link
Author

Hi @Court72

Can't we do anything? It's been 3 weeks already. Thank you

@Court72
Copy link
Contributor

Court72 commented May 8, 2024

@ggailey777

Can you review the proposed changes?

When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

@bledeamarius
Copy link
Author

@ggailey777 Can you please have a look at the PR? Thanks.

@AnnaMHuff
Copy link
Contributor

PR review team emailed author today

@v-dirichards
Copy link
Contributor

I sent an email to the content owner today.

@MicrosoftDocs/public-repo-pr-review-team

@v-dirichards
Copy link
Contributor

@ggailey777

Can you review this old PR and determine whether it needs to be closed or merged?

@MicrosoftDocs/public-repo-pr-review-team

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Updates Azure Functions Service Bus trigger documentation to include the required subscriptionName parameter for Python v1 functions. This fixes a documentation gap that was causing containerized Azure Functions to fail during startup.

  • Adds missing subscriptionName parameter to the function.json configuration example
  • Updates the descriptive text to mention both topicName and subscriptionName parameters

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

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.

5 participants