-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat(component): Add location & role to test data #35
Conversation
This updates the `Test component` to contain locations and roles for testing our viewer & editor.
To avoid regression after EasyDynamics/oscal-react-library#728 is merged in, this tests that locations and roles display properly. It will require changes from EasyDynamics/oscal-demo-content#35 for the tests to pass.
What did you actually change? The diff is a mess here. |
"roles" : [ { | ||
"id" : "creator", | ||
"title" : "Document creator", | ||
"short-name" : "creator", | ||
"description" : "Creates documents describing the system" | ||
}, { | ||
"id" : "contact", | ||
"title" : "Contact" | ||
} ], | ||
"locations" : [ { | ||
"uuid" : "54e8923b-56d2-46f1-bb97-9c6f734d9d9e", | ||
"title" : "NIST", | ||
"address" : { | ||
"addr-lines" : [ "100 Bureau Drive", "Mail Stop 8970" ], | ||
"city" : "Gaithersburg", | ||
"state" : "MD", | ||
"postal-code" : "20899-8970" | ||
}, | ||
"email-addresses" : [ "itl_inquiries@nist.gov" ], | ||
"telephone-numbers" : [ { | ||
"type" : "office", | ||
"number" : "301-975-8616" | ||
} ], | ||
"urls" : [ "https://www.nist.gov/", "https://www.nist.gov/itl/ssd/contact-us" ], | ||
"remarks" : "Working with industry and academia to enhance economic security and improve our quality of life." | ||
}, { | ||
"uuid" : "6c01ae87-5410-41af-87cd-3253dd7ccfc7", | ||
"address" : { } | ||
} ], |
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.
What did you actually change? The diff is a mess here.
@kylelaker This is actual change. Saving it in the Editor
changed the formatting.
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.
For roles
, I added a short-name
& description
to Document Creator.
There was previously no locations
.
To avoid regression after EasyDynamics/oscal-react-library#728 is merged in, this tests that locations and roles display properly. It will require changes from EasyDynamics/oscal-demo-content#35 for the tests to pass.
To avoid regression after EasyDynamics/oscal-react-library#728 is merged in, this tests that locations and roles display properly. It will require changes from EasyDynamics/oscal-demo-content#35 for the tests to pass. --------- Co-authored-by: Kyle Laker <klaker@easydynamics.com>
This updates the
Test component
to contain locations and rolesfor testing our viewer & editor.