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

add check before printing receipt items #19342

Merged
merged 2 commits into from
Jun 18, 2021
Merged

Conversation

squassina
Copy link
Contributor

@squassina squassina commented Jun 18, 2021

add check before printing the items in files:

  • sdk/formrecognizer/azure-ai-formrecognizer/samples/sample_recognize_receipts.py
  • sdk/formrecognizer/azure-ai-formrecognizer/samples/sample_recognize_receipts_from_url.py

this closes issue #19341

@kristapratico
Copy link
Member

@squassina looks great! Thank you for the quick PR. Could I ask that you add this check to the async samples as well?

sdk/formrecognizer/azure-ai-formrecognizer/samples/async_samples/sample_recognize_receipts_async.py
sdk/formrecognizer/azure-ai-formrecognizer/samples/async_samples/sample_recognize_receipts_from_url_async.py

@squassina
Copy link
Contributor Author

Done!

Copy link
Member

@kristapratico kristapratico left a comment

Choose a reason for hiding this comment

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

Awesome, thanks again for the contribution!

@kristapratico kristapratico merged commit 708bca1 into Azure:main Jun 18, 2021
@squassina squassina deleted the fix-19341 branch June 25, 2021 18:08
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-python that referenced this pull request Jun 7, 2022
update-datafactory-readme-typescript.md (Azure#19342)
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.

2 participants