[GitHub] [trafficcontrol] ocket8888 commented on a change in pull request #6115: Add `cdn.conf` option for enabling new role based permission

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

ocket8888 commented on a change in pull request #6115:
URL: https://github.com/apache/trafficcontrol/pull/6115#discussion_r690581664



##########
File path: docs/source/admin/traffic_ops.rst
##########
@@ -477,6 +477,12 @@ This file deals with the configuration parameters of running Traffic Ops itself.
 	.. versionadded:: 5.0
 		This is an optional boolean value to enable the handling of the "If-Modified-Since" HTTP request header. Default: false
 
+:role_based_permissions: Toggle whether or not to use Role Based Permissions.
+
+	.. versionadded:: 7.0
+		The blueprint can be seen :pr:`{{5848}}`

Review comment:
       I just wrapped it with `{{` and `}}` to show that it was a value to be replaced. This should just be
   ```rst
   :pr:`5848`
   ```

##########
File path: docs/source/admin/traffic_ops.rst
##########
@@ -477,6 +477,12 @@ This file deals with the configuration parameters of running Traffic Ops itself.
 	.. versionadded:: 5.0
 		This is an optional boolean value to enable the handling of the "If-Modified-Since" HTTP request header. Default: false
 
+:role_based_permissions: Toggle whether or not to use Role Based Permissions.
+
+	.. versionadded:: 7.0

Review comment:
       I'm fine with this for now, but note it'll need to change if we do a 6.1




-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@trafficcontrol.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org