Skip to content

Conversation

@jbrichau
Copy link

Description

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Checklist

My PR contains...

  • No code changes (src/ is unmodified: changes to documentation, CI, metadata, etc.)
  • Dependency changes (any modification to dependencies in package.json)
  • Bug fixes (non-breaking change which fixes an issue)
  • Improvements (misc. changes to existing features)
  • Features (non-breaking change which adds functionality)

My changes...

  • are breaking changes to a public API (config options, System API, major UI change, etc).
  • are breaking changes to a private API (Redux, component props, utility functions, etc.).
  • are breaking changes to a developer API (npm script behavior changes, new dev system dependencies, etc).
  • are not breaking changes.

Documentation

  • My changes do not require a change to the project documentation.
  • My changes require a change to the project documentation.
  • If yes to above: I have updated the documentation accordingly.

Automated tests

  • My changes can not or do not need to be tested.
  • My changes can and should be tested by unit and/or integration tests.
  • If yes to above: I have added tests to cover my changes.
  • If yes to above: I have taken care to cover edge cases in my tests.
  • All new and existing tests passed.

Ron and others added 30 commits December 14, 2018 20:53
* add tests

* compute index before triggering URL load

* bonus: improve urls topbar label
* Update LICENSE

* Update README.md
* add test cases

* update tag deeplinking implementation

* MOAR test updates

* update operation-tag.jsx
…i#5051)

This makes it easier to style them individually or retrieve the
elements in user scripts.
…#5120)

* Added markdown for header descriptions as per specifications:

https://github.com/OAI/OpenAPI-Specification/blob/master/versions/3.0.0.md#header-object
which subclasses:
https://github.com/OAI/OpenAPI-Specification/blob/master/versions/3.0.0.md#parameterObject

https://swagger.io/specification/#headerObject
which subclasses
https://swagger.io/specification/#parameterObject

* update headers.jsx

* Update headers.jsx

Fixing test error due to the `description` variable having been removed

* Update headers.jsx
…er-api#5142)

* add failing tests
* standardize parameter keying
* validateParam test migrations
* migrate test cases to new pattern
* disambiguate name/in ordering in `body.body` test cases
* `name+in`=> `{in}.{name}`
* consider allowEmptyValue parameter inclusion in runtime validation
* use config object for all validateParam options
* drop isXml flag from validateParams
…api#5157)

* add no-cache option for smaller images

* move flag immediately after `apk`

adjusting to be as close to the example as possible: https://github.com/gliderlabs/docker-alpine/blob/master/docs/usage.md#disabling-cache
…s selector (via swagger-api#5159)

* fix: handle malformed global tags array in taggedOperations

* handle non-array global tags as well

* update test imports

* remove stray brackets
…via swagger-api#5161)

* fix: don't trigger `url` remote document load if `urls` is provided

* better test block title
…api#5166)

* coerce multipart initial property values to string

* add tests
… (via swagger-api#5170)

* `schema` -> `structural`

* drop strip-instance transformer
mbaeuerle and others added 30 commits January 9, 2020 12:45
* Use official license text

* Update LICENSE

Co-authored-by: Ron <ron@swagger.io>
…pi#5737)

Co-authored-by: kyle shockey <kyleshockey@gmail.com>
…gger-api#5316)

* clears authentications when logout is clicked

* tests the headers sent in the network request

* adds test for multiple api keys

* refactors tests to extract common uses

* correct test message description

Co-authored-by: kyle shockey <kyleshockey@gmail.com>
…ger-api#5901)

Co-authored-by: kyle shockey <kyleshockey@gmail.com>
* Feature: Render OAS3 parameter type formats

* Add more test cases to parameter-row

* Update test/mocha/components/parameter-row.jsx

Co-Authored-By: Helen Kosova <hkosova@users.noreply.github.com>

* Update src/core/components/parameter-row.jsx

Co-authored-by: Helen Kosova <hkosova@users.noreply.github.com>
Co-authored-by: kyle shockey <kyleshockey@gmail.com>
Co-authored-by: Renovate Bot <bot@renovateapp.com>
Co-authored-by: kyle shockey <kyle.shockey1@gmail.com>
Co-authored-by: Renovate Bot <bot@renovateapp.com>
…#5095)

Co-authored-by: kyle shockey <kyle.shockey1@gmail.com>
…agger-api#5196)

* Added optional isShownKey prop to Componenent to allow overriding

* Removed unneccasry  before is isShwonKey

* Added PropTypes to isShownKey

* Added isShownKey to <OperationWrapper/> and removed it from <Operation />

* Removed isShwonKey prop it from <Operation />

* revert package-lock.json

Co-authored-by: kyle shockey <kyleshockey@gmail.com>
* fix: enable online validator badge

since online validator badge now can handle OAS3 specs
Co-authored-by: Renovate Bot <bot@renovateapp.com>
* Fix basic-auth.jsx: do not require password in UI

password is not PropTypes.string.isRequired, but the markup wrongly makes it mandatory

* Do not require password input in http-auth.jsx

This is used by the Authorize modal. Not sure when the similar code from basic-auth.jsx is used.

Co-authored-by: kyle shockey <kyle.shockey1@gmail.com>
…er-api#5376)

* support for supportedSubmitMethods property in react component

* update react README

Co-authored-by: Radek <radek.miazio@skysanner.net>
Co-authored-by: kyle shockey <kyle.shockey1@gmail.com>
This is a force-pushed squash of two PR merges (swagger-api#5919, swagger-api#5940) that were
formerly present on master as individual commits.

Co-Authored-By: kyle shockey <kyleshockey@gmail.com>
* prevent log warning for missing getComponent in production

eslint-plugin-react@7.19.0

* ft: new optional config parameter for getComponent

* Update src/core/plugins/view/root-injects.jsx

Co-Authored-By: kyle shockey <kyleshockey@gmail.com>

* Update src/core/plugins/view/root-injects.jsx

Co-Authored-By: kyle shockey <kyleshockey@gmail.com>

* Update src/core/plugins/view/root-injects.jsx

Co-Authored-By: kyle shockey <kyleshockey@gmail.com>

* Update src/core/json-schema-components.jsx

Co-Authored-By: kyle shockey <kyleshockey@gmail.com>

* prevent log warning for missing getComponent in production

* `npm audit fix`

* `npm install --save-dev chromedriver@80.0.1`

* `npm audit fix`

* update cypress

* Revert "update cypress"

This reverts commit c0404d4.

Co-authored-by: Timothy Lai <timothy.lai@gmail.com>
Co-authored-by: Tim Lai <tim-lai@users.noreply.github.com>
Co-authored-by: Kyle Shockey <shockey@users.noreply.github.com>
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.