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

Fix deletion confirmation message change in RHOAI 2.9.0 Dashboard #1301

Merged

Conversation

jiridanek
Copy link
Member

SeleniumLibrary . Page Should Contain Type ${item_title} to confirm deletion.

@jiridanek jiridanek requested a review from FedeAlonso March 15, 2024 08:51
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@jiridanek jiridanek added this to the 2.9.0 milestone Mar 15, 2024
Copy link
Contributor

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass %
448 0 0 448 100

@jstourac jstourac marked this pull request as draft March 15, 2024 08:56
@jstourac
Copy link
Member

This is supposed to be against 2.9 release. Please don't merge until we decide whether we'll branch 2.8 already.

@jiridanek
Copy link
Member Author

jiridanek commented Mar 15, 2024

@jstourac I set Milestone to 2.9.0. Is it still appropriate to have the PR as draft, when the changes in it are actually complete? I understand it is a practical safety measure, but still it feels wrong to me.

Maybe we should have a version.txt in the repository and a PR check that compares with that. So these ahead-of-time PRs would have failed CI, to indicate they should not be merged. Meanwhile, I guess I can accept the Draft.

@jiridanek jiridanek self-assigned this Mar 15, 2024
@jstourac
Copy link
Member

Okay, obviously different understanding. Personally I don't mind having PRs as drafts even if they are complete since it's safer than the do-not-merge label. But feel free to override me, I believe I may be just sole in this opinion.

@jstourac jstourac marked this pull request as ready for review March 15, 2024 15:26
@jiridanek jiridanek added the enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes) label Mar 15, 2024
@jiridanek jiridanek merged commit 2740c7a into red-hat-data-services:master Mar 17, 2024
12 checks passed
@jiridanek jiridanek deleted the jd_confirm_delete_modal branch March 17, 2024 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants