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

Frameworks #386

Closed
wants to merge 2 commits into from
Closed

Frameworks #386

wants to merge 2 commits into from

Conversation

hjacobs
Copy link
Contributor

@hjacobs hjacobs commented Jan 12, 2017

First goal to decouple Connexion from Flask: extract calls to Flask into the new Framework interface.

Addresses #380

** WORK IN PROGRESS - DO NOT MERGE **

PR opened for discussion cc @garbas

@dutradda
Copy link
Contributor

@hjacobs are you working on it now?
I am thinking to code to decouple flask from the validators classes.
I am creating a AioHttpFramework class and I stuck on validators. Can I decouple this things and the all things that I will need to create the AioHttpFramework?
I am thinking to create two simple namedtuple for Request/Response to wrap from on frameworks. This is necessary because all the frameworks have differents interfaces, and the connexion needs to have it own request/response interface.
So the Framework class should has two more abstract methods: _request_decorator and _response_decorator. This methods will do the conversion from frameworks request/response types to connexion types.

@rafaelcaricio
Copy link
Collaborator

@dutradda You can move forward with that.

@rafaelcaricio
Copy link
Collaborator

For now we are moving forward with #394. Closing this.

@jmcs jmcs removed the in progress label Feb 17, 2017
@rafaelcaricio rafaelcaricio deleted the frameworks branch April 4, 2017 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants