Conversation
- bumped swagger to 2.4.14 - updated docraptor.yaml - updated script/build to use generated build.sh for heavy lifting - updated libraries used in script/build - added System libraries when running tests Co-authored-by: Alex Overbeck <[email protected]> Co-authored-by: Nathan Acuff <[email protected]>
Why is this change needed? -------------------------- The strict parameter was causing some funny business. For whatever reason having a default value on a non-required parameter caused issues during instantiation with named parameters. The valid values for it are "none" and "html" How does it address the issue? ------------------------------ Since it's not a required parameter we removed the default value. We also fixed the valid values for the strict parameter to match the api. https://docraptor.com/documentation/api#api_strict Thirdly we moved pipeline down after the required fields so the important fields are listed first. Co-authored-by: Alex Overbeck <[email protected]>
Also switch to dependent libraries that swagger pulls in
The .swagger-codegen-ignore file tells swagger which files to leave alone during codegen. If big changes are made it would be worth letting swagger regenerate these but for the most part what we already have there is correct. Co-authored-by: Alex Overbeck <[email protected]>
- updated existing examples for swagger c# changes - created hosted document examples
- Added JsonSubTypes dependency - Updated example to match named parameter changes - verified and removed fixes for generated client section
- Renamed DocStatus > AsyncDocStatus - Created DocStatus that mirrors all the fields of AsyncDocStatus Co-authored-by: Alex Overbeck <[email protected]>
This reverts commit 6ab079e. This commit was designed to increase backwards compatibility but after further discussion it was decided that the added complexity was less desirable. Additionally there are already breaking changes in this new version that will need address and this would be a minor one.
jason-o-matic
approved these changes
Jul 31, 2020
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
V2.0.0 - Hosted Documents
See below for important changes
As a paid add-on, DocRaptor can provide long-term, publicly-accessible hosting for your documents. This allows you to provide a URL to your end users, third party tools like Zapier and Salesforce, or anyone else. We'll host the document on your behalf at a completely unbranded URL for as long as you want, or within the limits you specify.
When you create a hosted document, DocRaptor will generate your document as usual. Asynchronous documents work as usual, except that the download URL in the final status response will be an unbranded domain. Synchronous hosted document requests respond with a JSON object rather than a binary blob.
The JSON will look like this:
{ "download_id": "123-456-abc", "download_url": "http://<<unbranded domain>>/download/123-456-abc", "number_of_pages": 1 }The download URL is publicly-accessible and doesn't require authentication.
By default, hosted documents do not have limits on downloads or hosting time, though you may pass additional parameters to the document generation call to set your own limits. Learn more about the specific options in the hosted API documentation.
Important API Changes
Constructor Argument Casing
The casing on the
Docmodel constructor arguments have been changed to reflect best reflect C# coding style best practices. This means that when we build aDocthe arguments are now camelCased where the used to be PascalCase.This is a breaking change.
Before:
After:
DocStatus
To avoid confusion between the status of an asynchronous document and a hosted document the
AsyncDocStatusmodel has been removed in favor of a shared,DocStatusmodel. This means that when you create an async document or hosted document both will return aDocStatus.This is a breaking change.
Before:
When checking the status of an async document docraptor.GetAsyncDocStatus returned an AsyncDocStatus.
After:
Now when checking the status of an async document
docraptor.GetAsyncDocStatusreturns aDocStatus.Checking the status of a hosted async document works the same way. docraptor.GetAsyncDocStatus returns a DocStatus.
Creating a synchronous hosted document works in a similar way, but because it is synchronous the
docraptor.CreateHostedDocmethod will return aDocStatusupon completion.Example
Here is a quick example on how to create a hosted document with C#.