search

HTTP-APIs / hydra-openapi-parser

OpenAPI to Hydra parser
MIT License
9 stars 9 forks source link

Update parser in accordance with changes made at hydra-python-core #19

Open vddesai1871 opened 5 years ago

vddesai1871 commented 5 years ago

I'm submitting a

Current Behaviour:

The current parser uses old Hydra terms which are replaced since long, as we have updated hydra-python-core with developments made at Hydra-CG spec, outdated parser will create a HydraDoc incompatible with hydra-python-core and hydrus.

Expected Behaviour:

The parser should be updated with respect to changes made at hydra-python-core and thus on parsing it should create a HydraDoc compatible with latest Hydra Spec.

Steps to reproduce:

Do you want to work on this issue?

YES

roufiq commented 3 years ago

no follow up?

chrizandr commented 3 years ago

A lot of changes in both hydra-spec and the openapi spec have been made since this project was last updated. This issue is only regarding updates with hydra-python-core, but the broader issue of updating in accordance to the spec needs to be done before this issue can be taken up.

farazkhanfk7 commented 3 years ago

If no one is working on it I would like to take this one.

amalthundiyil commented 2 years ago

@chrizandr Would it be acceptable to resolve this issue by fixing the parser at its current state first and then addressing the "broader issue" by creating a separate issue?

farazkhanfk7 commented 2 years ago

@amal-thundiyil The current version of hydra-openapi-parser is outdated. It uses old Hydra terms which are no longer in use. Also, hydra-python-core has gone through lots of developments since the last time parser was updated, as a result it throws error and returns an incompatible HydraDoc. Many functions in hydra-openapi-parser are missing parameters. The parser should be updated according to the changes in hydra-python-core and Hydra Specs. And yes, you can create a separate issue for that as well.