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

SWC-6618: show projects associated with an AR #5247

Merged
merged 1 commit into from
Jan 3, 2024

Conversation

hallieswan
Copy link
Contributor

Shows a list of projects associated with an AR. Will need to bump SRC before merging.

ACT

Managed - ManagedAR_ACT
Managed Team - ManagedTeamAR_ACT
Self Sign - SelfSignAR_ACT

Non-ACT

Managed - ManagedAR_NonACT
Managed Team - ManagedTeamAR_NonACT
Self Sign - SelfSignAR_NonACT

lazyLoadHelper.setIsConfigured();
view.setAccessRequirementID(ar.getId().toString());
isACTMemberAsyncHandler.isACTActionAvailable(isACT -> {
view.setAccessRequirementIDVisible(isACT);
view.setCoveredEntitiesHeadingVisible(isACT);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only show the "Covered Entities" heading when the projects list and entities table are shown, i.e. when the user is ACT

<bh:Hr addStyleNames="margin-top-5 margin-bottom-5" />
</bh:Div>
<bh:Div
ui:field="accessRequirementRelatedProjectsListContainer"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

swc.css has list styles that override the MUI list styles. Add a class name to the div so that styles can be re-specified.

Comment on lines +3892 to +3894
// swc.css list styles override MUI list styles
// Re-specify MUI list style here
.accessRequirementRelatedProjectsListContainer > div {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Override the swc.css list styles, so that styles match MUI

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works for now knowing how challenging it is to get MUI to work with bootstrap, normalize, and anything custom we have in SWC

@hallieswan hallieswan merged commit 6f689ab into Sage-Bionetworks:develop Jan 3, 2024
2 checks passed
@hallieswan hallieswan deleted the SWC-6618 branch January 3, 2024 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants