- 
                Notifications
    You must be signed in to change notification settings 
- Fork 300
WIP: initial openapi schema generator implementation #669
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
Conversation
| TODOs: 
 | 
| This is quite a lengthy PR so not sure when I will find time to look at it. But I assume you are still working on it? As this is another optional feature we should fix #674 first to cleanly define optional deps needed by this feature. | 
| Any news about this? I see the only missing checks are: 
 Anything I could perhaps help with? | 
| 
 @arielpontes Thanks for the offer to help. There's a bit more to do than that, including the TODOs referenced above and the need to do #674 first. Perhaps you can start on that? There's also an additional optional package which is django-oauth-toolkit (used in generateschema to add oauth  This code is based on DRF 3.10 pre-release which has since had a couple of minor releases -- currently at 3.10.2 -- so there may be some additional changes needed. I wanted to wait until DRF 3.10 settled down a bit and hope to have some time in the next week or two to get back to this, so it would really help if you (or @sliverc) could knock out #674. Thanks! | 
| Codecov Report
 @@            Coverage Diff             @@
##           master     #669      +/-   ##
==========================================
- Coverage   95.96%   93.24%   -2.73%     
==========================================
  Files          54       58       +4     
  Lines        2728     3166     +438     
==========================================
+ Hits         2618     2952     +334     
- Misses        110      214     +104
 Continue to review full report at Codecov. 
 | 
This reverts commit 72091ba.
| I'm closing this in lieu of a new PR that isn't all messed up with the various merges from master. | 
Fixes #604
Description of the Change
Extends DRF >= 3.10's generateschema to produce a jsonapi-formatted OAS schema document.
Checklist
CHANGELOG.mdupdated (only for user relevant changes)AUTHORSThis is currently conditionalized to test for DRF >=3.10 so none of the test cases are executed, even with drfmaster (since 3.10 is not yet released), but, if you comment out those tests then it should work with drfmaster. Still some build failures with obsolete py27, py34, django111, etc. which should go away when #664 is merged.