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

Dynamic configuration API documentation refactor #15077

Closed

Conversation

ektravel
Copy link
Contributor

@ektravel ektravel commented Oct 4, 2023

Reorganization of the Dynamic configuration API page

This PR:

  • Adds headings to the page to separate methods
  • Adds curl and HTTP code tabs
  • Adds request and response examples
  • Improves descriptions of methods
  • Updates links

This PR has:

  • been self-reviewed.

Preview

https://dynamic-config-api-doc-refactor-qi7p.vercel.app/docs/latest/api-reference/dynamic-configuration-api


Screen Shot 2023-10-04 at 8 58 09 AM

@kfaraz
Copy link
Contributor

kfaraz commented Oct 4, 2023

@ektravel , could you please include some more details about the changes made in the PR description?

docs/api-reference/dynamic-configuration-api.md Outdated Show resolved Hide resolved
docs/api-reference/dynamic-configuration-api.md Outdated Show resolved Hide resolved
docs/api-reference/dynamic-configuration-api.md Outdated Show resolved Hide resolved
@ektravel
Copy link
Contributor Author

ektravel commented Oct 5, 2023

Closing this PR in favor of #15098

@ektravel ektravel closed this Oct 5, 2023
@ektravel ektravel deleted the dynamic-config-api-doc-refactor branch October 5, 2023 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants