-
Notifications
You must be signed in to change notification settings - Fork 11
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
Display System Short Name and Description for SSP #611
Display System Short Name and Description for SSP #611
Conversation
Add to the System Characteristics section of the SSP Viewer to display the system's short name next to the system name, and display the description underneath the system name.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor style and linting requests
{/* Render the system's short name in parentheses next to the name, | ||
but only if the short name exists. */} | ||
{props.systemCharacteristics["system-name-short"] && | ||
" (" + props.systemCharacteristics["system-name-short"] + ")"} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
{/* Render the system's short name in parentheses next to the name, | |
but only if the short name exists. */} | |
{props.systemCharacteristics["system-name-short"] && | |
" (" + props.systemCharacteristics["system-name-short"] + ")"} | |
{ | |
// Render the system's short name in parentheses next to the | |
// name, but only if the short name exists. | |
props.systemCharacteristics["system-name-short"] && | |
`(${props.systemCharacteristics["system-name-short"]})` | |
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This comment may still upset the linter -- but I want to use this (${})
with string interpolation instead of string concatenation. So this may need another npm run lint:fix
once it's been applied.
</Grid> | ||
<Grid item xs={6}> | ||
<Typography variant="body2"> | ||
{props.systemCharacteristics["description"]} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
{props.systemCharacteristics["description"]} | |
{props.systemCharacteristics.description} |
Address linter errors.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add to the System Characteristics section of the SSP Viewer to display the system's short name next to the system name, and display the description underneath the system name.
The short name functionality can be tested using FedRAMP's SSP Template.