-
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
EGRC-406: Display of Control Parameter Constraints in Profiles #65
Conversation
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 found a few minor things we could potentially tweak to clean up code a bit, but otherwise everything makes sense and works!
index.toString() | ||
); | ||
})} | ||
{props.modificationDisplay} |
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.
Something to clean up in the future - Since this is its own component now, we probably don't need to pass in modificationDisplay, which is also its own discrete component. Might be cleaner to display them both from within ControlPart.
Reviewing now. |
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.
All the implemented features work. However, I got a bug when running locally on Chrome: sometimes it will take upwards of ~15 seconds to reload the page when using the reload button and FedRamp profile.
arrow
You can see the resulting dot badge and tooltip on hover by navigating to the Profile Viewer and viewing one of the FedRAMP profiles: