Skip to content

Nested allOf#47

Open
trejjam wants to merge 1 commit into
paypal:mainfrom
trejjam:patch-1
Open

Nested allOf#47
trejjam wants to merge 1 commit into
paypal:mainfrom
trejjam:patch-1

Conversation

@trejjam
Copy link
Copy Markdown

@trejjam trejjam commented Apr 25, 2026

Hi, you do not have issues open. So I am contacting you here.

You managed to duplicate allOf at several places:
image

It breaks code generators, since this is kind of broken.

The correct version is without duplicated allOf:

      "order_authorize_response": {
        "title": "Order Authorize Response",
        "description": "The order authorize response.",
        "type": "object",
        "allOf": [
          {
            "$ref": "#/components/schemas/activity_timestamps"
          },
          {
            "title": "activity_timestamps"
          },
          {
            "type": "object",
            "title": "Order Authorize Response",
            "description": "The order authorize response.",
            "properties": {
            }
          }
        ]
      }

@trejjam trejjam changed the title Duplicated up allOf Duplicated nested allOf Apr 25, 2026
@trejjam trejjam changed the title Duplicated nested allOf Nested allOf Apr 25, 2026
@romanloveislife1999-hub
Copy link
Copy Markdown

ROMAN@1

@mynewworld225-cmyk
Copy link
Copy Markdown

mynewworld225-cmyk commented May 20, 2026 via email

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.

3 participants