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

Added support for showExtensions and $ref for Responses #1406

Merged
merged 4 commits into from
Dec 9, 2021

Conversation

shelbys
Copy link
Contributor

@shelbys shelbys commented Oct 9, 2020

If you appreciate this contribution, please add a hacktoberfest-accepted Label to the PR, so it'll be tracked

Copy link
Member

@RomanHotsiy RomanHotsiy left a comment

Choose a reason for hiding this comment

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

It looks good, the only question is if the test message is corret?

@@ -31,5 +31,12 @@ describe('Models', () => {
const resp = new ResponseModel(parser, 'default', true, {}, opts);
expect(resp.type).toEqual('error');
});

test('should be error if showExtensions is true', () => {
Copy link
Member

Choose a reason for hiding this comment

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

should be error?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops! I missed updating the first half. ;)

@RomanHotsiy
Copy link
Member

Thanks for your contribution.

I will merge it after the release today so it will land into the next release.

@shelbys
Copy link
Contributor Author

shelbys commented Oct 15, 2020

Thanks much, including for the tagging

@shelbys
Copy link
Contributor Author

shelbys commented Oct 27, 2020

@RomanHotsiy - Please merge this when you get a chance to unblock #1434 and #1435

@RomanHotsiy
Copy link
Member

Hey @shelbys!

I am very sorry for the 1-year delay! 🙈
We've been busy with other projects and I missed this PR (and many more others 😢 ).

We now have more resources and time to work on Redoc so this PR will be merged today.

@Oprysk Oprysk merged commit f82eb53 into Redocly:master Dec 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants