Alfred Jonsson

> @AffeJonsson > > A conflict occurred when I merged the following PR. Could you please check again and correct it? > > https://github.com/anymaniax/orval/pulls?q=is%3Apr+is%3Aclosed Merged master and did a small...

Maybe wait with this until #1199 is complete? I assume we might need to change this when it is completed anyway

> it would make sense to regenerate samples > > example in screenshot has `overrideResponse` twice 🤔 > > ``` > ...overrideResponse > // some faker > ...overrideResponse > ```...

Seems I broke tests with this, fixing...

> @AffeJonsson Please let me know when this is ready for me to review 👍 It's ready!

Putting the discriminator on the base class also works. But having the discriminator on the response is valid according to the spec: https://swagger.io/docs/specification/data-models/inheritance-and-polymorphism/

I realize there is two ways this could be solved, but I'm not sure which is the way to go. Either we add the discriminated property to the types, or...

If you want to test different response types, you can use the `generateEachHttpStatus` config option in `output.mock`: ```ts output: { mock: { type: "msw", generateEachHttpStatus: true } } ``` With...