Skip to content

feat(MFA): Add docs for Multi-Factor Authentication #2895

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

Merged
merged 7 commits into from
Apr 30, 2025

Conversation

Meggielqk
Copy link
Collaborator

No description provided.

@Meggielqk Meggielqk added this to the 5.9.0 milestone Apr 21, 2025
@Meggielqk
Copy link
Collaborator Author

Still need to update 2 screenshots

@Meggielqk Meggielqk marked this pull request as ready for review April 22, 2025 05:15
@Meggielqk Meggielqk requested review from zmstone and ysfscream April 22, 2025 06:29
@ysfscream ysfscream requested a review from Kinplemelon April 22, 2025 07:43
zmstone
zmstone previously approved these changes Apr 22, 2025
ysfscream
ysfscream previously approved these changes Apr 23, 2025
@ysfscream
Copy link
Member

@Kinplemelon If there are no further issues related to the Dashboard, please approve and merge this PR. Thank you very much.

@Meggielqk Meggielqk requested a review from Copilot April 23, 2025 05:49
Copy link

@Copilot 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

This PR adds comprehensive documentation for the new Multi-Factor Authentication (MFA) feature in EMQX Dashboard.

  • Adds MFA documentation in both Chinese (zh_CN) and English (en_US).
  • Updates admin guide files (both Chinese and English) to include links to the MFA documentation.
  • Updates the directory navigation (dir.yaml) to reference the new MFA documentation.

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
zh_CN/multi-factor-authn/multi-factor-authentication.md New MFA documentation added in Chinese
zh_CN/admin/admin-guide.md Added MFA link in the admin guide
en_US/multi-factor-authn/multi-factor-authentication.md New MFA documentation added in English
en_US/admin/admin-guide.md Added MFA link in the admin guide
dir.yaml Included MFA documentation in navigation directory

Kinplemelon
Kinplemelon previously approved these changes Apr 23, 2025
@Meggielqk Meggielqk dismissed stale reviews from Kinplemelon, ysfscream, and zmstone via f440021 April 23, 2025 07:17
Kinplemelon
Kinplemelon previously approved these changes Apr 28, 2025
@Meggielqk Meggielqk requested a review from Copilot April 30, 2025 03:31
Copilot

This comment was marked as duplicate.

@zmstone zmstone merged commit c3d2d4f into release-5.9 Apr 30, 2025
7 checks passed
@zmstone zmstone deleted the r59-multi-factor-authn branch April 30, 2025 12:01
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.

4 participants