-
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
feat(metadata): Support viewing revisions #781
Conversation
d559974
to
0013fc7
Compare
ac5c553
to
14ddf62
Compare
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.
I am loving these TS reworks as you're going through. THANK YOU for that! I am good with the design and approach here. I'd love to make that table a more shared file...
Thank you for the tests!
I do wonder if we should have a display for a revision that isn't just a table row... but I guess we don't have anywhere to display that for now, so maybe we should just add it when we need it.
@@ -60,7 +60,7 @@ export interface PropertyFilter { | |||
export function propWithName( | |||
props: Property[] | undefined, | |||
name: string, | |||
ns: string | undefined | |||
ns?: string | undefined |
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.
oof good catch!
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 looks good to me! Nice work @tuckerzp
I'd love to make that table a more shared file...
I totally agree. My current issue also touches on the same table styles. I could see these being handled at a more global scale.
Creates a modal with a table to view metadata revision history.
Closes #705
Testing
http://localhost:3000/catalog/?url=https://raw.githubusercontent.com/EasyDynamics/oscal-demo-content/tuckerzp/revision-example/catalogs/basic-test-catalog.json