Skip to content

Conversation

@SimonRichardson
Copy link

@SimonRichardson SimonRichardson commented May 9, 2022

Changes:

ErrorResponse for /api/v2/template/apply

The following adds documentation about a new feature that issues an
error message if the template is too large. Currently, this targets only
cloud, but we should consider this for OSS. I'll open this up for
discussion, if in doubt I'll move this to cloud only as a divergent API.


  • Fix the bug, that if you have two error codes that have the same body, then it creates a $ref type that prevents it from parsing correctly.

@SimonRichardson SimonRichardson added the documentation Improvements or additions to documentation label May 9, 2022
Changes:

  ErrorResponse for /api/v2/template/apply

The following adds documentation about a new feature that issues an
error message if the template is too large. Currently, this targets only
cloud, but we should consider this for OSS. I'll open this up for
discussion, if in doubt I'll move this to cloud only as a divergent API.
@SimonRichardson SimonRichardson force-pushed the document-template-too-large branch from e517592 to 6f5b71a Compare May 11, 2022 09:28
The following just ensures that we don't inline the reference for errors
that have the same body but different error code. This is shame, but I
can see why.

The fix is simple, just create a new template reference that duplicates
the TemplateSummaryError.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant