[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #3456: Fix the parameter name of deliveryserviceserver in document

[GitBox <gi...@apache.org>]

ocket8888 commented on a change in pull request #3456: Fix the parameter name of deliveryserviceserver in document
URL: https://github.com/apache/trafficcontrol/pull/3456#discussion_r272251625
 
 

 ##########
 File path: docs/source/api/deliveryserviceserver.rst
 ##########
 @@ -107,9 +107,9 @@ Assign a set of one or more servers to a :term:`Delivery Service`
 
 Request Structure
 -----------------
-:deliveryService: The integral, unique identifier of the :term:`Delivery Service` to which the servers identified in the ``servers`` array will be assigned
-:replace:         If ``true``, any existing assignments for a server identified in the ``servers`` array will be overwritten by this request
-:servers:         An array of integral, unique identifiers for servers which are to be assigned to the :term:`Delivery Service` identified by ``deliveryService``
+:dsId:    The integral, unique identifier of the :term:`Delivery Service` id to which the servers identified in the ``servers`` array will be assigned
 
 Review comment:
   we don't need to specify twice that this is an identifier, "integral, unique identifier" already does that, so adding "id" is redundant.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services