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

✨ Set application delete as danger kebab menu item (#1829) #1840

Merged
merged 1 commit into from
Apr 12, 2024

Conversation

sjd78
Copy link
Member

@sjd78 sjd78 commented Apr 11, 2024

Part of #1318
Resolves: https://issues.redhat.com/browse/MTA-2261
Backport-of: #1829

Comment on lines -1015 to -1025
{
isAriaDisabled:
application.migrationWave !== null,
tooltipProps: {
content:
application.migrationWave !== null
? t(
"message.cannotDeleteApplicationsAssignedToMigrationWave"
)
: "",
},
Copy link
Collaborator

@mguetta1 mguetta1 Apr 11, 2024

Choose a reason for hiding this comment

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

@sjd78 did you delete lines 1015-1025 in purpose?
If the reason is to backport also #1803, please modify applicationDropdownItems as well

Copy link
Collaborator

Choose a reason for hiding this comment

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

Looks like #1803 should not be backported yet, fix version is MTA 7.1.0
https://issues.redhat.com/browse/MTA-2225

Copy link
Member Author

Choose a reason for hiding this comment

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

The bot failed to create the backport/cherry-pick so I did it manually without thinking about it too much. I'll go double check the original PR and update. There was no intention to also include any other change.

Copy link
Member Author

Choose a reason for hiding this comment

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

Removed this erroneous change in the latest push.

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh wait -- I see the nuance now. I'll keep the pre-1803 version of the delete button but move it to the newer location, and add the danger style

@sjd78 sjd78 force-pushed the backport_1829_release-0.3 branch from ab644f8 to 234de06 Compare April 11, 2024 18:20
Part of konveyor#1318

Resolves: https://issues.redhat.com/browse/MTA-2261

---------

Signed-off-by: Maayan Hadasi <mguetta@redhat.com>
Co-authored-by: Scott Dickerson <sdickers@redhat.com>
Signed-off-by: Scott J Dickerson <sdickers@redhat.com>
@sjd78 sjd78 force-pushed the backport_1829_release-0.3 branch from 234de06 to a683f4b Compare April 11, 2024 18:27
@sjd78 sjd78 requested a review from mguetta1 April 11, 2024 18:27
Copy link
Collaborator

@mguetta1 mguetta1 left a comment

Choose a reason for hiding this comment

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

LGTM

@sjd78 sjd78 merged commit 43cd5e9 into konveyor:release-0.3 Apr 12, 2024
4 checks passed
@sjd78 sjd78 deleted the backport_1829_release-0.3 branch April 19, 2024 17:09
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