Ciac 16249 improve docusign documentation and add get-event command#43770
Ciac 16249 improve docusign documentation and add get-event command#43770lironcohen272 merged 10 commits intomasterfrom
Conversation
🤖 AI-Powered Code Review AvailableYou can leverage AI-powered code review to assist with this PR! Available Commands:
|
|
🤖 Analysis started. Please wait for results... |
🤖 AI Review DisclaimerThis review was generated by an AI-powered tool and may contain inaccuracies. Please be advised, and we extend our sincere apologies for any inconvenience this may cause. |
marketplace-ai-reviewer
left a comment
There was a problem hiding this comment.
Hi! Thanks for contributing to the Docusign pack. I've reviewed your changes and left a few notes to help polish the integration. The main items to address include using .get() for safer dictionary access in the Python code, updating the new command name to follow standard conventions, and adding the missing required fields to the pack metadata. Thanks again for your hard work!
Additionally, please address the following file-level notes:
Packs/Docusign/Integrations/Docusign/README.md: Ensure README is SDK-generated.Packs/Docusign/pack_metadata.json: - The mandatorycreatedkey is missing from the pack metadata. Please add it with the pack's creation date.- The vendor name ('Docusign') must be included in the
keywordslist. Currently, the list is empty.
@kamalq97 please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.
kamalq97
left a comment
There was a problem hiding this comment.
Looks great! Minor changes needed.
… to integration content access issue
|
Validate summary Verdict: PR can be force merged from validate perspective? ✅ |
fixes: https://jira-dc.paloaltonetworks.com/browse/CIAC-16249%20Improve%20Docusign%20documentation
Description
Improve Docusign documentation and add get-event command