Skip to content

doc: ldap settings#503

Merged
wingwing2005 merged 3 commits into
mainfrom
main-hlb
May 31, 2026
Merged

doc: ldap settings#503
wingwing2005 merged 3 commits into
mainfrom
main-hlb

Conversation

@huolibo
Copy link
Copy Markdown
Contributor

@huolibo huolibo commented May 30, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 30, 2026 02:47
@huolibo huolibo requested a review from wingwing2005 May 30, 2026 02:47
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds documentation for LDAP integration, including configuration, user synchronization, and login flows, in both Chinese and English. The review feedback highlights minor inconsistencies between the two language versions: the Chinese documentation is missing a bullet point regarding the retention of deleted LDAP users, and the English version is missing a concluding sentence about password storage. Both comments provide helpful suggestions to align the translations.

Comment thread docs/14-administration/04-user-management.md
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds LDAP configuration and behavior documentation to the User Management administration docs (EN + ZH), covering connection settings, user synchronization, and LDAP-based login flow.

Changes:

  • Documented LDAP connection fields and their meanings in Admin Console → User Management → LDAP.
  • Added user sync behavior, including attribute mapping and matching/update rules.
  • Documented the LDAP login flow and credential handling.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
i18n/en/docusaurus-plugin-content-docs/current/14-administration/04-user-management.md Adds an English LDAP section with configuration, sync, and login behavior.
docs/14-administration/04-user-management.md Adds a Chinese LDAP section mirroring the LDAP configuration/sync/login documentation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/14-administration/04-user-management.md Outdated
Comment thread docs/14-administration/04-user-management.md
Copilot AI review requested due to automatic review settings May 30, 2026 03:05
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

| **Search Filter** | Yes | Additional LDAP search criteria, for example `(objectClass=inetOrgPerson)`. |
| **Roles** | Yes | Default roles automatically assigned to synchronized users. Each role can have its own configurable set of accessible elements. |

Click save after completing the configuration. If the password field displays a masked value (`********`), the existing password is retained. Clear the field to remove the password.
| **搜索过滤器** | 是 | 额外的 LDAP 搜索条件,例如 `(objectClass=inetOrgPerson)`。|
| **角色** | 是 | 同步用户时自动分配的默认角色。每个角色可配置可访问的元素范围。 |

配置完成后点击保存。保存时密码字段若显示为掩码(`********`),表示保留原密码不变;清空则移除密码。
| **Search Filter** | Yes | Additional LDAP search criteria, for example `(objectClass=inetOrgPerson)`. |
| **Roles** | Yes | Default roles automatically assigned to synchronized users. Each role can have its own configurable set of accessible elements. |

Click save after completing the configuration. If the password field displays a masked value (`********`), the existing password is retained. Clear the field to remove the password.
| **搜索过滤器** | 是 | 额外的 LDAP 搜索条件,例如 `(objectClass=inetOrgPerson)`。|
| **角色** | 是 | 同步用户时自动分配的默认角色。每个角色可配置可访问的元素范围。 |

配置完成后点击保存。保存时密码字段若显示为掩码(`********`),表示保留原密码不变;清空则移除密码。
Copy link
Copy Markdown
Contributor

@wingwing2005 wingwing2005 left a comment

Choose a reason for hiding this comment

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

LGTM

@wingwing2005 wingwing2005 merged commit e293a0d into main May 31, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants