search

Mermade / openapi-gui

GUI / visual editor for creating and editing OpenAPI / Swagger definitions
https://mermade.github.io/openapi-gui/
MIT License
1.42k stars 170 forks source link

fixed the bug, removed 'required:false' it was causing additional prroperty error. #62

Closed manmohan-99 closed 4 years ago

manmohan-99 commented 4 years ago

there was a hard-coded required field in method.js which was causing additional property error.

MikeRalphson commented 4 years ago

Ah, good spot, but it looks like the required: false should be up one level (in rb).

manmohan-99 commented 4 years ago

required field is already there in bottom of rb which we can set true or false, no issue with that after removing that hard-coded required it is working fine(checked on swagger editor).

MikeRalphson commented 4 years ago

Thanks.