Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Server tutorial has outdated syntax #333

Closed
czechboy0 opened this issue Oct 16, 2023 · 0 comments · Fixed by #338
Closed

Server tutorial has outdated syntax #333

czechboy0 opened this issue Oct 16, 2023 · 0 comments · Fixed by #338
Labels
area/documentation Improvements or additions to documentation. kind/bug Feature doesn't work as expected. size/S Small task. (A couple of hours of work.)
Milestone

Comments

@czechboy0
Copy link
Contributor

czechboy0 commented Oct 16, 2023

https://swiftpackageindex.com/apple/swift-openapi-generator/0.3.0/tutorials/swift-openapi-generator/serverswiftpm

.text doesn't build, should be .plainText. We should search the repo for other occurrences.

And the string needs to be wrapped in HTTPBody.

@czechboy0 czechboy0 added area/documentation Improvements or additions to documentation. kind/bug Feature doesn't work as expected. size/S Small task. (A couple of hours of work.) labels Oct 16, 2023
@czechboy0 czechboy0 added this to the 1.0 milestone Oct 16, 2023
tib added a commit to tib/swift-openapi-generator that referenced this issue Oct 23, 2023
@czechboy0 czechboy0 linked a pull request Oct 23, 2023 that will close this issue
czechboy0 pushed a commit that referenced this issue Oct 24, 2023
This pull request aims to fix the server tutorial sample code.
Sample snippets updated to use Swift 5.9 and Swift OpenAPI generator
tools 0.3.0+
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Improvements or additions to documentation. kind/bug Feature doesn't work as expected. size/S Small task. (A couple of hours of work.)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant