Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@edewit
Copy link
Contributor

@edewit edewit commented Oct 3, 2023

fixes: #22600

@edewit edewit requested a review from a team as a code owner October 3, 2023 13:06
@ghost ghost added the team/ui label Oct 3, 2023
@cypress
Copy link

cypress bot commented Oct 3, 2023

2 flaky tests on run #9425 ↗︎

0 527 48 0 Flakiness 2

Details:

Merge 378415e into 7f9ea3a...
Project: Keycloak Admin UI Commit: 790acdfd42 ℹ️
Status: Passed Duration: 12:30 💡
Started: Oct 17, 2023 7:23 AM Ended: Oct 17, 2023 7:35 AM
Flakiness  clients_test.spec.ts • 1 flaky test • chrome

View Output Video

Test Artifacts
Clients test > Accessibility tests for clients > Check a11y violations on load/ clients list tab Test Replay Output Screenshots
Flakiness  realm_settings_general_tab_test.spec.ts • 1 flaky test • chrome

View Output Video

Test Artifacts
Realm settings general tab tests > Test all general tab switches Test Replay Output Screenshots

Review all test suite changes for PR #23667 ↗︎

edewit added a commit to edewit/keycloak that referenced this pull request Oct 16, 2023
back to having the server check this
now that we have keycloak#23667 there are no issues with spaces
jonkoops
jonkoops previously approved these changes Oct 16, 2023
Copy link
Contributor

@jonkoops jonkoops left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for this @edewit!

@jonkoops
Copy link
Contributor

I've moved the utility function to a dedicated file, looks like the React Router imports were causing some issues with the utility tests, this should mitigate that.

@jonkoops jonkoops enabled auto-merge (squash) October 16, 2023 10:02
@jonkoops jonkoops merged commit 13207aa into keycloak:main Oct 17, 2023
@edewit edewit deleted the issue-22600 branch October 17, 2023 08:37
edewit added a commit to edewit/keycloak that referenced this pull request Oct 17, 2023
jonkoops added a commit that referenced this pull request Oct 17, 2023
edewit added a commit that referenced this pull request Oct 17, 2023
back to having the server check this
now that we have #23667 there are no issues with spaces
edewit added a commit to edewit/keycloak that referenced this pull request Oct 17, 2023
back to having the server check this
now that we have keycloak#23667 there are no issues with spaces
jonkoops pushed a commit that referenced this pull request Oct 17, 2023
back to having the server check this
now that we have #23667 there are no issues with spaces
@stianst stianst mentioned this pull request Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Keycloak admin v2 theme js mapper adding issue

2 participants