Skip to content
This repository was archived by the owner on Sep 14, 2022. It is now read-only.

"Hello World"-example should match its spec ( Fixes #476 ) #475

Merged
merged 10 commits into from
Feb 22, 2017
Merged

"Hello World"-example should match its spec ( Fixes #476 ) #475

merged 10 commits into from
Feb 22, 2017

Conversation

meisterlampe
Copy link
Contributor

@meisterlampe meisterlampe commented Feb 19, 2017

Hi,

I just started experimenting with swagger and followed the steps in the Readme.
When I generated a C#-Client for the "Hello World"-Example-Server, it wasn't working out of the box, because the server just returns "Hello, stranger" instead of "{ "message": "Hello, stranger" } as specified in the swagger-spec.
I first created an issue in the swagger-codegen-repo (see swagger-api/swagger-codegen#4816), which was the wrong place for reporting :)

This is only a minor bug I know, but the example should work out of the box. This pull request should make the example match its spec again. It should fix: #476

@meisterlampe meisterlampe changed the title "Hello World"-example should match its spec "Hello World"-example should match its spec ( Fixes #476 ) Feb 20, 2017
@meisterlampe
Copy link
Contributor Author

meisterlampe commented Feb 20, 2017

Sorry for all the commits. I had to fix every single file using the web frontend O:)
Feel free to squash them into a single one.

@fehguy fehguy merged commit 1f01f6a into swagger-api:master Feb 22, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Hello World" Example doesn't follow the spec file
2 participants