-
Notifications
You must be signed in to change notification settings - Fork 362
Add documentation for 3.4.0 #741
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
base: main
Are you sure you want to change the base?
Conversation
* add multiple header parsing to "what's new" * add multiple header parsing to client-responses.md
fix: KTOR-8880
* KTOR-9150 Add documentation for partial HTML response * fix formatting, grammar, and language in server-responses.md
* fix: add Whats New section * Apply suggestions from code review Co-authored-by: Vik Nikolova <[email protected]> * KTOR-9185 Add a new topic and project sample for the HttpRequestLifecycle plugin --------- Co-authored-by: Vik Nikolova <[email protected]>
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
* KTOR-9175 Add api key auth documentation --------- Co-authored-by: Vik Nikolova <[email protected]>
AlejandraPedroza
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Huge work! I left some suggestions. Please check:
- External links (especially those pointing to ktorio repo on GitHub), as they appear as page not found.
- Code comments that should be in 3rd person verb, above the code line, and other criteria listed in our style guide.
- Consistency across plugin names, which in some cases appear in monospaced fonts and in some cases they not.
- Functions that may be missing
()next to their names.
codeSnippets/snippets/server-http-request-lifecycle/src/main/kotlin/com/example/Application.kt
Outdated
Show resolved
Hide resolved
codeSnippets/snippets/server-http-request-lifecycle/src/main/kotlin/com/example/Application.kt
Outdated
Show resolved
Hide resolved
codeSnippets/snippets/server-http-request-lifecycle/src/test/kotlin/ApplicationTest.kt
Outdated
Show resolved
Hide resolved
AlejandraPedroza
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Huge work! I left some suggestions. Please check:
- External links (especially those pointing to ktorio repo on GitHub), as they appear as page not found.
- Code comments that should be in 3rd person verb, above the code line, and other criteria listed in our style guide.
- Consistency across plugin names, which in some cases appear in monospaced fonts and in some cases they not.
- Functions that may be missing
()next to their names.
…#742) * add a what's new entry for dispatcher config for native client engines * fix broken API link and add a new one for engine configs
AlejandraPedroza
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the changes. LGTM 🚀
New topics:
whats-new-340.md)server-http-request-lifecycle.md)server-api-key-auth.md)New sections:
client-auth.md)client-responses.md)server-compression.md)server-html-dsl.md)client-plugins,client-default-request)server-responses.md)Improved formatting, grammar and language:
server-responses.md)server-basic-auth.md,server-bearer-auth)New code samples:
server-http-lifecycle