-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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 organizations enabled/disabled capability #29573
Add organizations enabled/disabled capability #29573
Conversation
js/apps/admin-ui/maven-resources/theme/keycloak.v2/admin/messages/messages_en.properties
Outdated
Show resolved
Hide resolved
model/infinispan/src/main/java/org/keycloak/models/cache/infinispan/RealmAdapter.java
Outdated
Show resolved
Hide resolved
model/infinispan/src/main/java/org/keycloak/models/cache/infinispan/UserCacheSession.java
Show resolved
Hide resolved
model/jpa/src/main/java/org/keycloak/organization/jpa/JpaOrganizationProvider.java
Show resolved
Hide resolved
model/storage-private/src/main/java/org/keycloak/storage/UserStorageManager.java
Show resolved
Hide resolved
server-spi-private/src/main/java/org/keycloak/models/utils/ModelToRepresentation.java
Outdated
Show resolved
Hide resolved
model/infinispan/src/main/java/org/keycloak/models/cache/infinispan/RealmAdapter.java
Outdated
Show resolved
Hide resolved
...sts/base/src/test/java/org/keycloak/testsuite/organization/admin/OrganizationMemberTest.java
Show resolved
Hide resolved
9ce8a77
to
39e9c39
Compare
8ad8a34
to
385bcad
Compare
Closes keycloak#28804 Signed-off-by: vramik <vramik@redhat.com>
…disabled Closes keycloak#29483 Signed-off-by: vramik <vramik@redhat.com>
385bcad
to
27fd37b
Compare
Why would we do this like this for organizations, why wouldn't it work like the other features? When you add the feature to the start script the UI will add the "Organization section" to the UI? |
…sResource Closes keycloak#29574 Signed-off-by: vramik <vramik@redhat.com>
Thank you @edewit
I'm not sure I completely understand the comment. Could you please elaborate/rephrase it? Also what do you mean by "start script"? |
@edewit I hope the flag introduced here is more clear now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving based on review of @pedroigor and @sguilhen
Closes #28804
Closes #29483
Closes #29574