Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Conversation

@ocket8888
Copy link
Contributor

This PR fixes invalid JSON syntax in the response body example documentation of the /deliveryservices/{{ID}}/urlkeys endpoint.

It also addresses docs build warnings that currently exist on master.


Which Traffic Control components are affected by this PR?

  • Documentation

What is the best way to verify this PR?

Build and read the docs

If this is a bugfix, which Traffic Control versions contained the bug?

Quite possibly all of them since the introduction of the endpoint.

PR submission checklist

  • This PR doesn't need tests
  • This PR is documentation
  • This PR doesn't need a CHANGELOG.md entry
  • This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY

@ocket8888 ocket8888 added bug something isn't working as intended documentation related to documentation low impact affects only a small portion of a CDN, and cannot itself break one labels Aug 19, 2022
Copy link
Contributor

@rawlinp rawlinp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rawlinp rawlinp merged commit 87445b7 into apache:master Aug 24, 2022
@ocket8888 ocket8888 deleted the docs/fix-syntax-error branch August 24, 2022 17:09
zrhoffman pushed a commit to zrhoffman/trafficcontrol that referenced this pull request Oct 2, 2022
* Fix invalid JSON syntax in API docs

* Fix extant docs build warnings

(cherry picked from commit 3f040fa)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug something isn't working as intended documentation related to documentation low impact affects only a small portion of a CDN, and cannot itself break one

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants