APIs-guru / google-discovery-to-swagger

Script for converting Google Discovery format into OpenAPI (swagger) 3.0
MIT License
57 stars 16 forks source link

Media upload and other bugs #9

Closed chingor13 closed 7 years ago

chingor13 commented 7 years ago

This PR fixes 3 issues:

  1. Converts resource methods that supportsMediaUpload into 3 separate paths (original, simple, and resumable).
  2. Some paths in Google discovery docs have a plus in the url. Example: Datastore has v1/{+name}. For these, we simply remove the +.
  3. A few Google discovery docs provide default boolean values that are not strings. Example: Drive, Gmail, and Youtube.
MikeRalphson commented 7 years ago

Many thanks for this PR @chingor13 - would you consider submitting it as 3 (or more) separate PRs, one for each issue addressed, in case we need to revert any of them in the future?

chingor13 commented 7 years ago

@MikeRalphson Sure, will do.