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

feat(THEEDGE-3758): show both tabs when adding systems to group #2120

Merged
merged 2 commits into from
Dec 14, 2023

Conversation

ldjebran
Copy link
Contributor

Show both tabs when adding systems to group, when adding conventional systems, conventional tab should be selected by default and when adding immutable systems, immutable tab should be selected by default.

FIXES: https://issues.redhat.com/browse/THEEDGE-3758

group-details-add-systems

@ldjebran ldjebran requested a review from a team as a code owner December 11, 2023 12:48
@ldjebran ldjebran changed the title feat(InventoryGroupDetails): THEEDGE-3758-show both tabs when adding systems. feat(InventoryGroupDetails): show both tabs when adding systems. Dec 11, 2023
@ldjebran ldjebran changed the title feat(InventoryGroupDetails): show both tabs when adding systems. feat(InventoryGroupDetails): show both tabs when adding systems to group. Dec 11, 2023
@ldjebran ldjebran changed the title feat(InventoryGroupDetails): show both tabs when adding systems to group. feat(THEEDGE-3758): show both tabs when adding systems to group. Dec 11, 2023
@ldjebran ldjebran changed the title feat(THEEDGE-3758): show both tabs when adding systems to group. feat(THEEDGE-3758): show both tabs when adding systems to group Dec 11, 2023
@ldjebran ldjebran force-pushed the group-details-add-systems-tabs branch from 8e075ba to 9056bf0 Compare December 11, 2023 14:32
Copy link
Contributor

@mkholjuraev mkholjuraev left a comment

Choose a reason for hiding this comment

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

@ldjebran thank you! Codewise looks good and works as described. However, let's add some coverage tests to make sure we avoid any possible future issues before merging.

src/components/GroupSystems/GroupSystems.js Outdated Show resolved Hide resolved
Show both tabs when adding systems to group, when adding conventional systems, conventional tab should be selected by default and when adding immutable systems, immutable tab should be selected by default.
FIXES: https://issues.redhat.com/browse/THEEDGE-3758
@ldjebran ldjebran force-pushed the group-details-add-systems-tabs branch from 9056bf0 to 8add871 Compare December 14, 2023 13:41
@ldjebran
Copy link
Contributor Author

/retest

Copy link
Contributor

@mkholjuraev mkholjuraev left a comment

Choose a reason for hiding this comment

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

LGTM! Works as described

@gkarat
Copy link
Contributor

gkarat commented Dec 14, 2023

/retest

@gkarat gkarat enabled auto-merge (squash) December 14, 2023 15:18
@gkarat gkarat added the enhancement New feature or request label Dec 14, 2023
@gkarat
Copy link
Contributor

gkarat commented Dec 14, 2023

/retest

@gkarat gkarat merged commit fb656c6 into RedHatInsights:master Dec 14, 2023
2 checks passed
@gkarat
Copy link
Contributor

gkarat commented Dec 14, 2023

🎉 This PR is included in version 1.62.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants