Skip to content

Added proper navigation in the docs#878

Merged
bharathk08 merged 1 commit intodevfrom
newfix/DOC-686
Dec 16, 2025
Merged

Added proper navigation in the docs#878
bharathk08 merged 1 commit intodevfrom
newfix/DOC-686

Conversation

@bharathk08
Copy link
Contributor

@bharathk08 bharathk08 commented Dec 16, 2025

Summary by CodeRabbit

  • Documentation
    • Updated Password Policy configuration guide with clarified navigation steps through the User Profile menu
    • Revised Two-Factor Authentication setup instructions with new navigation pathway and updated screenshots
    • Enhanced User Settings documentation with improved menu structure and expanded feature options

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 16, 2025

Walkthrough

Documentation updates across three security and user settings guide files restructure navigation instructions and step sequences to reflect a new User Profile menu-based access pattern in the bottom-left corner, replacing previous navigation paths. Step numbers and image references are correspondingly updated.

Changes

Cohort / File(s) Summary
Documentation Navigation Flow Updates
src/pages/docs/configuration/security/password-policy.md, src/pages/docs/configuration/security/setup-two-factor-authentication.md, src/pages/docs/configuration/user-settings.md
Restructured navigation instructions across three documentation files to reflect new User Profile menu access pattern. Password Policy and 2FA setup flows now begin by accessing the User Profile menu in the bottom-left corner. User Settings section expanded to include Profile, Change Password, and 2FA options. Step sequences renumbered to account for additional initial navigation step. Image references updated to align with revised step flows. Wording refined for consistency (e.g., "Click" vs. "Select" directives).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

  • Documentation content reordering and image reference updates following a consistent navigation pattern across related files
  • No code changes; straightforward instructional text restructuring
  • Pattern consistency across all three files reduces per-file reasoning complexity

Possibly related PRs

  • Added docs fo User & Organization Settings #877: Main PR updates the same documentation pages (user-settings.md, password-policy.md, setup-two-factor-authentication.md), with this PR appearing as a related refinement or follow-up to navigation flow updates.

Suggested reviewers

  • Santhosh-testsigma
  • Nandhini4501

Poem

🐰 A user's journey, refined with care,
Menu buttons now in corners fair,
Steps renumbered, paths made clear,
Two-Factor, passwords—no more fear!
Documentation hops along so bright,

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: updating documentation to reflect proper navigation paths across multiple user settings and authentication pages.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch newfix/DOC-686

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4d15728 and 5498eb7.

📒 Files selected for processing (3)
  • src/pages/docs/configuration/security/password-policy.md (1 hunks)
  • src/pages/docs/configuration/security/setup-two-factor-authentication.md (1 hunks)
  • src/pages/docs/configuration/user-settings.md (3 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: CI
🔇 Additional comments (5)
src/pages/docs/configuration/security/password-policy.md (1)

34-51: Navigation restructuring looks good.

The updated steps properly guide users from the User Profile menu to Organization Settings > Password Policy. Step numbering is correct, image references follow consistent conventions, and the flow is logical. This aligns well with the navigation pattern updates in the other documentation files.

src/pages/docs/configuration/user-settings.md (3)

43-49: User Settings introduction is clear and well-organized.

The initial navigation step and options list are consistent with the PR's navigation pattern changes. The bulleted list of three configurable areas (Profile, Change Password, 2FA) provides good visual clarity.


55-61: Profile section steps are clear and complete.

The three-step flow (Select Profile, Edit fields, Update) is straightforward. Image references align with the instructions, and the subsection numbering starting at 1 is standard for multi-section documentation.


67-74: Change Password section follows consistent patterns.

The three-step sequence properly guides users through password change (Select option, Enter passwords, Update). Instructions are specific about required fields, and image references support each step.

src/pages/docs/configuration/security/setup-two-factor-authentication.md (1)

37-59: 2FA setup flow is logical and well-structured.

The 8-step sequence properly guides users through the complete 2FA setup process, from initial menu access through final verification. The step progression and descriptions are clear.


5. Click **Continue**. This will open the **Verify Authenticator App** dialog.
6. Click **Continue**. This will open the **Verify Authenticator App** dialog.
![Continue after QR Code](https://s3.amazonaws.com/static-docs.testsigma.com/new_images/projects/Updated_Doc_Images/Contiune_After_QR_Code.png)
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

🏁 Script executed:

# Check if the file exists and read line 52 with context
head -n 60 src/pages/docs/configuration/security/setup-two-factor-authentication.md | tail -n 20

Repository: testsigmahq/testsigma-docs

Length of output: 1474


🏁 Script executed:

# Search for the typo "Contiune" in the repository
rg "Contiune" --type md

Repository: testsigmahq/testsigma-docs

Length of output: 289


🏁 Script executed:

# Search for the corrected version "Continue_After_QR_Code" to see if it appears anywhere
rg "Continue_After_QR_Code" --type md

Repository: testsigmahq/testsigma-docs

Length of output: 52


Fix typo in image filename.

Line 52 references an image with a typo: "Contiune_After_QR_Code.png" should be "Continue_After_QR_Code.png" to match the naming pattern of adjacent image references in the same file (Continue_with_2FA.png, Continue_after_entering_Authenticator_Code.png).

Apply this diff:

-   ![Continue after QR Code](https://s3.amazonaws.com/static-docs.testsigma.com/new_images/projects/Updated_Doc_Images/Contiune_After_QR_Code.png)
+   ![Continue after QR Code](https://s3.amazonaws.com/static-docs.testsigma.com/new_images/projects/Updated_Doc_Images/Continue_After_QR_Code.png)

Ensure the actual image file on S3 matches the corrected filename.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
![Continue after QR Code](https://s3.amazonaws.com/static-docs.testsigma.com/new_images/projects/Updated_Doc_Images/Contiune_After_QR_Code.png)
![Continue after QR Code](https://s3.amazonaws.com/static-docs.testsigma.com/new_images/projects/Updated_Doc_Images/Continue_After_QR_Code.png)
🤖 Prompt for AI Agents
In src/pages/docs/configuration/security/setup-two-factor-authentication.md
around line 52, the embedded image filename has a typo
("Contiune_After_QR_Code.png"); update the Markdown reference to
"Continue_After_QR_Code.png" to match the naming pattern used elsewhere, and
verify that the S3 object at that exact path/name exists (or upload/rename the
file on S3 to match the corrected filename) so the image loads correctly.

@bharathk08 bharathk08 merged commit 95a61f9 into dev Dec 16, 2025
3 of 4 checks passed
@bharathk08 bharathk08 deleted the newfix/DOC-686 branch December 16, 2025 10:58
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.

2 participants