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

Add common KW Open "Get Started" For App Name #1352

Merged
merged 1 commit into from
Apr 8, 2024

Conversation

manosnoam
Copy link
Contributor

@manosnoam manosnoam commented Apr 8, 2024

  • Replacing duplicated code to open the "Get Started" of Explorer cards
  • Fix the Selenium error while trying to select an Explorer card: Element is not clickable at point. Other element would receive the click

Before this fix, our tests failed on:

image

With this fix it is resolved:

image

Copy link
Contributor

github-actions bot commented Apr 8, 2024

Robot Results

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

@manosnoam manosnoam added verified This PR has been tested with Jenkins enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes) labels Apr 8, 2024
@manosnoam manosnoam requested review from bdattoma and jstourac April 8, 2024 09:23
@manosnoam manosnoam force-pushed the click_card branch 2 times, most recently from 406e445 to cdfd8a0 Compare April 8, 2024 10:22
jstourac
jstourac previously approved these changes Apr 8, 2024
- Replacing duplicated code to open the "Get Started" of Explorer cards
- Fix the Selenium error while trying to select an Explorer card:
`Element is not clickable at point. Other element would receive the click`

Signed-off-by: manosnoam <[email protected]>
Copy link

sonarqubecloud bot commented Apr 8, 2024

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

@manosnoam manosnoam requested a review from jstourac April 8, 2024 10:37
@manosnoam manosnoam merged commit 21908f7 into red-hat-data-services:master Apr 8, 2024
10 checks passed
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) verified This PR has been tested with Jenkins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants