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

fix(control): change "Guidance" to "Discussion" #683

Merged
merged 1 commit into from
Feb 28, 2023

Conversation

kylelaker
Copy link
Contributor

"Discussion" is the terminology used in the SP 800-53 PDF document,
whereas "Guidance" is the term used within the OSCAL document. According
to the 800-53rev5 Example from NIST, guidance was retained from
an earlier version of the data and tools reading and displaying it are
free to make appropriate adjustments. To keep terminology consistent
between the viewed version of the document and the expected PDF, we
should use "Discussion". Within code, we should continue using
"Guidance" (at least for now) because this is the terminology used in
the source code.

Examples

Control List View

image

Control Discussion Modal

image

"Discussion" is the terminology used in the SP 800-53 PDF document,
whereas "Guidance" is the term used within the OSCAL document. According
to the [800-53rev5 Example][1] from NIST, `guidance` was retained from
an earlier version of the data and tools reading and displaying it are
free to make appropriate adjustments. To keep terminology consistent
between the viewed version of the document and the expected PDF, we
should use "Discussion". Within code, we should continue using
"Guidance" (at least for now) because this is the terminology used in
the source code.

[1]: https://pages.nist.gov/OSCAL/concepts/layer/control/catalog/sp800-53rev5-example/
@Bronstrom Bronstrom merged commit af56319 into develop Feb 28, 2023
@Bronstrom Bronstrom deleted the kyle/guidance-discussion branch February 28, 2023 20:43
@kylelaker kylelaker added the bug Something isn't working label Jun 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants