Skip to content

fix(settings): restore LDAP periodic sync control in Community Edition - #5827

Merged
w940853815 merged 1 commit into
devfrom
pr@dev@fix_settings_ldap-periodic
Oct 10, 2026
Merged

w940853815 merged 1 commit into
devfrom
pr@dev@fix_settings_ldap-periodic

Conversation

@fit2bot

@fit2bot fit2bot commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

LDAP sync settings reused the shared periodic checkbox configuration, whose license check hides the control in Community Edition. Restore a plain checkbox so administrators can enable or disable LDAP periodic synchronization from the UI. Keep the organization selector and LDAP HA license checks unchanged.

This fixes the hidden control reported in jumpserver/jumpserver#17684. The reported automatic reset on save was not reproduced.

Validation: yarn fix, yarn lint, and the production build passed on the latest dev base using Node 24 and existing installed dependencies. Targeted in-memory assertions confirmed CE/EE visibility, preserved true/false submission values, and unchanged organization/LDAP HA license checks. No test files are included. Live LDAP end-to-end verification was not performed.

@fit2bot
fit2bot requested a review from a team October 10, 2026 10:28
@sonarqubecloud

Copy link
Copy Markdown

@w940853815
w940853815 merged commit 686ddb1 into dev Oct 10, 2026
5 of 6 checks passed
@w940853815
w940853815 deleted the pr@dev@fix_settings_ldap-periodic branch October 10, 2026 10:29
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.

2 participants