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 new requests to docs #38

Merged
merged 2 commits into from
Jan 9, 2024
Merged

Add new requests to docs #38

merged 2 commits into from
Jan 9, 2024

Conversation

maxmati
Copy link
Contributor

@maxmati maxmati commented Jan 9, 2024

No description provided.

type: string
format: url
description: |
DA URLs to the documents to sign.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
DA URLs to the documents to sign.
DA URL to the documents to sign.

It's only one, even if it's an array of objects, right?

description: Password for password encrypted requests
responses:
200:
description: All pending signature requests for the file were cancelled.
Copy link
Contributor

Choose a reason for hiding this comment

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

We return 200 when all the requests were cancelled? Same below for /request/file/status. Can you explain?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy and paste issue from the place where it already was copy paste issue :D

@maxmati maxmati requested a review from ianaz January 9, 2024 12:11
@maxmati maxmati merged commit a9b13f3 into main Jan 9, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants