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.

Sign up for GitHub

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

Jump to bottom

Support for adding new params via update_api #642

Merged
merged 2 commits into from
Nov 15, 2018
Merged

Support for adding new params via update_api #642

merged 2 commits into from
Nov 15, 2018

Commits on Nov 9, 2018

  1. Support for adding new params via update_api 

    Before this patch, we only supported updating nested hashes. For example

      param :user, Hash do
        param :login, String
      end

and

      param :user, Hash do
        param :oauth, String
      end

has been merged properly. However when merging top-level params, such as

      param :login, String

and

      param :oauth, String

no changes were propagated.
    @iNecas
    iNecas committed Nov 9, 2018
    Configuration menu
    Copy the full SHA
    f7de142 View commit details
    Browse the repository at this point in the history

  2. Fix intermittent swagger test failures 

    In some Ruby implementations, we couldn't rely on specific order in the hash.
The new matcher implementation doesn't rely on the order.
    @iNecas
    iNecas committed Nov 9, 2018
    Configuration menu
    Copy the full SHA
    47058d2 View commit details
    Browse the repository at this point in the history