Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

CRDCDH-493 Org Management studies list tooltip fixes #187

Merged
merged 3 commits into from
Oct 31, 2023

Conversation

amattu2
Copy link
Member

@amattu2 amattu2 commented Oct 31, 2023

Overview

This PR resolves a bug ticket and implements requirement changes to the Organization Management list (table).

Change Details (Specifics)

  • When an organization has >0 studies assigned, show the first study's abbreviation (e.g. XY01)
  • If an organization has >1 studies assigned, append and other {count} to the study list
  • Swapped the click-based tooltip with a hover based tooltip

Related Ticket(s)

https://tracker.nci.nih.gov/browse/CRDCDH-493
https://tracker.nci.nih.gov/browse/CRDCDH-320

@amattu2 amattu2 marked this pull request as ready for review October 31, 2023 14:30
@amattu2 amattu2 added this to the MVP-2.0.0 (M2) milestone Oct 31, 2023
Copy link
Collaborator

@Alejandro-Vega Alejandro-Vega left a comment

Choose a reason for hiding this comment

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

LGTM!

@Alejandro-Vega Alejandro-Vega merged commit 0071910 into mvp-2 Oct 31, 2023
@Alejandro-Vega Alejandro-Vega deleted the amattu2/CRDCDH-493 branch October 31, 2023 14:52
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants