Misleading visibility_timeout, Required vs optional parameters, updates needed!
Vignesh-Microsoft opened this issue · comments
I see this correction on the sdk git but not on docs. Please update as needed.
The PR and commit:
Also, for update_message, pop_receipt and content don't need to be a required parameter. One place it says optional but another it says required. Details below:
WHY is it optional, because
-
pop_receipt is already part of the message object. pop_receipt is only required if message object is not there like in case of cli and powershell.
-
if we are issuing an update call for visibility_timeout alone, we do not need content as required parameter so it is optional.
Same for delete_message
Commit:
Please review and approve if it looks good.
Ported this to the SDK repo: Azure/azure-sdk-for-python#24397