Skip to content

Fix: Allow to define type fields with whole document or remote reference - #308

Merged
2 commits merged into
oapi-codegen:masterfrom
wtertius:allow_to_define_type_fields_throw_whole_document_reference
Mar 18, 2021
Merged

2 commits merged into
oapi-codegen:masterfrom
wtertius:allow_to_define_type_fields_throw_whole_document_reference

Conversation

@wtertius

@wtertius wtertius commented Mar 2, 2021

Copy link
Copy Markdown
Contributor

Fix: Allow to define type fields with whole document or remote reference

It fixes the following case:

$ cat main.yml:
  ...
      elements:
        type: array
        items:
          $ref: './schemas/items.yml'

$ cat ./schemas/items.yml
type: object
properties:
  id:
    type: integer
    format: int64
  name:
    type: string

@wtertius
wtertius force-pushed the allow_to_define_type_fields_throw_whole_document_reference branch 3 times, most recently from a72759c to 3fd3ca2 Compare March 9, 2021 18:29
…nce.

It fixes the following case:
$ cat main.yml:
  ...
      elements:
        type: array
        items:
          $ref: './schemas/items.yml'

$ cat ./schemas/items.yml
type: object
properties:
  id:
    type: integer
    format: int64
  name:
    type: string
@wtertius
wtertius force-pushed the allow_to_define_type_fields_throw_whole_document_reference branch from 3fd3ca2 to 0f1d198 Compare March 9, 2021 18:33
@ghost
ghost merged commit ff7eeab into oapi-codegen:master Mar 18, 2021
adrianpk pushed a commit to foorester/oapi-codegen that referenced this pull request Jan 16, 2024
…nce. (oapi-codegen#308)

It fixes the following case:
$ cat main.yml:
  ...
      elements:
        type: array
        items:
          $ref: './schemas/items.yml'

$ cat ./schemas/items.yml
type: object
properties:
  id:
    type: integer
    format: int64
  name:
    type: string

Co-authored-by: Lukiianchuk Dmitrii <d.lukiyanchuk@butik.ru>
Co-authored-by: Marcin Romaszewicz <47459980+deepmap-marcinr@users.noreply.github.com>
This pull request was closed.
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.

1 participant