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

[ui] Show more information when reviewing recommendations #953

Merged
merged 2 commits into from
Feb 26, 2025

Conversation

evamillan
Copy link
Contributor

This PR adds a list of the individual's usernames and email addresses to the cards in the "Review recommendations" modal to compare them more easily. It also adds more source icons to avoid grouping many of them into "other sources".

imagen

Fixes #952, #936

Copy link
Member

@sduenas sduenas left a comment

Choose a reason for hiding this comment

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

The PR looks great. I think it allows to figure out faster if the individuals are the same or not.
Just a minor commit before we merge it. Can you add a new changelog file to indicate new icons for the datasources have been added?

Adds the Font Awesome icons set, which includes icons for
Confluence, Jenkins, Phabricator and Meetup. To use them in
the VIcon component, their name must be prefaced by 'fa:'.
It also adds an svg icon for Gerrit.

Signed-off-by: Eva Millán <[email protected]>
Adds a 'detailed' prop to the IndividualCard component,
which shows a list of usernames and email addresses instead
of only the source icons, to be used on recommendations.

Signed-off-by: Eva Millán <[email protected]>
@evamillan evamillan force-pushed the improve-ui-recommendations branch from 255d0a4 to a9c3399 Compare February 26, 2025 15:59
@evamillan
Copy link
Contributor Author

Thanks @sduenas, I added the changelog file.

Copy link
Member

@sduenas sduenas left a comment

Choose a reason for hiding this comment

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

LGTM

@sduenas sduenas merged commit c2d340e into chaoss:main Feb 26, 2025
16 checks passed
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.

[FR] Review recommendations: Show details of both candidates
2 participants