Skip to content

Always use useIsFeatureEnabled() hook for feature detection#19589

Merged
jonkoops merged 1 commit into
keycloak:mainfrom
jonkoops:consistent-feature-detection
Apr 9, 2023
Merged

Always use useIsFeatureEnabled() hook for feature detection#19589
jonkoops merged 1 commit into
keycloak:mainfrom
jonkoops:consistent-feature-detection

Conversation

@jonkoops

@jonkoops jonkoops commented Apr 7, 2023

Copy link
Copy Markdown
Contributor

Ensures that feature detection is always done with the same hook, this avoids repeating the same code for feature detection and will make it easier to refactor in the future.

@jonkoops jonkoops force-pushed the consistent-feature-detection branch from c9ecd30 to 0d89759 Compare April 7, 2023 12:39
@cypress

cypress Bot commented Apr 7, 2023

Copy link
Copy Markdown

4 flaky tests on run #6687 ↗︎

0 513 47 0 Flakiness 4

Details:

Merge 0d89759 into 396e2ba...
Project: Keycloak Admin UI Commit: dc6c859798 ℹ️
Status: Passed Duration: 14:45 💡
Started: Apr 7, 2023 12:48 PM Ended: Apr 7, 2023 1:03 PM
Flakiness  realm_settings_tabs_test.spec.ts • 1 flaky test

View Output Video

Test Artifacts
Realm settings tabs tests > Accessibility tests for realm settings > Check a11y violations on user registration tab/ default roles sub tab Output Screenshots
Flakiness  client_authorization_test.spec.ts • 1 flaky test

View Output Video

Test Artifacts
Client authentication subtab > Should create a policy Output Screenshots
Flakiness  clients_saml_test.spec.ts • 1 flaky test

View Output Video

Test Artifacts
Clients SAML tests > SAML keys tab > should doesn't disable signature when cancel Output Screenshots
Flakiness  realm_test.spec.ts • 1 flaky test

View Output Video

Test Artifacts
Realm tests > should fail creating realm with empty name Output Screenshots

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@jonkoops jonkoops enabled auto-merge (squash) April 7, 2023 14:11
@jonkoops jonkoops merged commit bdb9c14 into keycloak:main Apr 9, 2023
@jonkoops jonkoops deleted the consistent-feature-detection branch April 11, 2023 09:03
jonkoops added a commit that referenced this pull request Apr 11, 2023
jonkoops added a commit that referenced this pull request Apr 11, 2023
jonkoops added a commit that referenced this pull request Apr 11, 2023
ssilvert pushed a commit that referenced this pull request Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants