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

Warning "Schema items must be present only for schemas of type 'array' " is promoted when adding a parameter through UI -- openapi: 3.0.2 #785

Closed
yuanhuiliu opened this issue May 29, 2019 · 15 comments
Labels

Comments

@yuanhuiliu
Copy link

"Schema items must be present only for schemas of type 'array' " is promoted when adding path parameters, query parameters or header parameters through UI. The screenshot of warning and code is as follows:
paramsError

The warning will disappear when updating anything in Open API source code and save it, such as description.

@yuanhuiliu yuanhuiliu changed the title Warning "Schema items must be present only for schemas of type 'array' " is promoted when adding a parameter through UI Warning "Schema items must be present only for schemas of type 'array' " is promoted when adding a parameter through UI -- openapi: 3.0.2 May 29, 2019
@Ranbato
Copy link

Ranbato commented Jun 11, 2019

I see this all the time as well. Let me know if you need a sample

@EricWittmann
Copy link
Member

A sample would be great, thanks.

@Ranbato
Copy link

Ranbato commented Jun 13, 2019

Test API.yaml.zip

This is the smallest example I can make. Note that after exporting the file it no longer shows the error in the UI, but if I delete and re-add the response the error shows up again.
It also shows up if I mark anything as required.

@EricWittmann
Copy link
Member

Thanks.

@EricWittmann
Copy link
Member

I can not reproduce this with the latest version of Apicurio. Admittedly a large number of changes have been made to both editing and validation, so it is possible that this has simply been fixed.

It's also possible I'm simply missing something. I'm going to close this for now, but I would very much appreciate it if someone could re-open it if the next release of Apicurio (will be done this week) still exhibits this problem.

@bj-mcduck
Copy link

@EricWittmann What's the latest version? I'm on the site right now and I'm experiencing this issue still.

@EricWittmann
Copy link
Member

EricWittmann commented Jun 19, 2019 via email

@bj-mcduck
Copy link

Any specific steps do you want me to record?

@EricWittmann
Copy link
Member

EricWittmann commented Jun 19, 2019 via email

@bj-mcduck
Copy link

bj-mcduck commented Jun 19, 2019

apicurio-schemaerror-sm
I tried sharing this longer and in higher resolution, but that didn't work out for me. :/

@yuanhuiliu
Copy link
Author

Add the screencast to reproduce this issue for the online version.
array-error

@EricWittmann
Copy link
Member

Thanks!

@EricWittmann
Copy link
Member

OK I have now been able to reproduce this (thanks for the videos!) in the online version (which is older) but cannot reproduce this with the latest version on master. I'm going to leave this closed for now and will be doing a new release today or tomorrow (which will result in the online version being updated). So hopefully this will be resolved. The data modeling, validation, and editing commands have all been massively changed, so I'm guessing we'll see other problems instead. :)

@yuanhuiliu
Copy link
Author

Thank, I will try the new version after it is released.

@EricWittmann
Copy link
Member

The new version has been released and is available at https://studio.apicur.io if you'd like to give it a try.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants