Skip to content

Docs: ACL rules location update#532

Merged
arseny-dragonflydb merged 1 commit into
mainfrom
docs/acl-rules-location-update
Jun 1, 2026
Merged

Docs: ACL rules location update#532
arseny-dragonflydb merged 1 commit into
mainfrom
docs/acl-rules-location-update

Conversation

@arseny-dragonflydb
Copy link
Copy Markdown
Contributor

  • Updates ACL Rules doc

@vercel
Copy link
Copy Markdown

vercel Bot commented May 31, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
documentation Ready Ready Preview, Comment May 31, 2026 5:07pm

Request Review

@qodo-free-for-open-source-projects
Copy link
Copy Markdown

Review Summary by Qodo

Update ACL Rules access instructions in datastores documentation

📝 Documentation

Grey Divider

Walkthroughs

Description
• Updates ACL Rules access instructions in datastores documentation
• Clarifies navigation path to Access menu before ACL Rules
Diagram
flowchart LR
  A["Click three-dot menu"] --> B["Select Access menu"]
  B --> C["Click ACL Rules"]
  C --> D["Edit ACL rules"]

Loading

Grey Divider

File Changes

1. docs/cloud/datastores.md 📝 Documentation +1/-1

Clarify ACL Rules access navigation path

• Updated instruction for accessing ACL Rules from "click **ACL Rules**" to "select **Access** ->
 **ACL Rules**"
• Clarifies the navigation path by explicitly mentioning the Access menu step
• Improves user guidance for locating ACL Rules configuration

docs/cloud/datastores.md


Grey Divider

Qodo Logo

@qodo-free-for-open-source-projects
Copy link
Copy Markdown

qodo-free-for-open-source-projects Bot commented May 31, 2026

Code Review by Qodo

🐞 Bugs (1) 📘 Rule violations (0)

Grey Divider


Advisory comments

1. UI path style inconsistent 🐞 Bug ⚙ Maintainability
Description
The ACL Rules navigation instruction introduces a third breadcrumb style (split-bold with ->),
while other Cloud docs mostly use a single bold breadcrumb with Access > .... Mixed notation makes
the docs harder to scan and maintain because the same kind of UI path is written multiple ways
across pages.
Code

docs/cloud/datastores.md[R205-207]

Evidence
The changed line uses split-bold + ->, while multiple other Cloud docs show the Access > ...
breadcrumb convention; there is also an existing Access -> ... instance, indicating inconsistency
that this PR further expands.

docs/cloud/datastores.md[205-210]
docs/cloud/sso/add-sso-connection.md[23-34]
docs/cloud/sso/domains.md[27-33]
docs/cloud/sso/okta.md[147-154]
docs/cloud/sso/okta.md[220-225]
docs/cloud/sso/onelogin.md[288-292]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The doc line uses `**Access** -> **ACL Rules**`, which differs from the more common Cloud-docs breadcrumb formatting (e.g., `**Access > SSO**`, `**Access > Users**`). This creates inconsistent UI path notation.

## Issue Context
Other Cloud SSO docs consistently use the `Access > ...` style, though there is at least one existing `Access -> ...` usage as well. This PR adds another variation (split bold segments + `->`).

## Fix Focus Areas
- docs/cloud/datastores.md[205-207]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

Qodo Logo

@arseny-dragonflydb arseny-dragonflydb merged commit 37fd841 into main Jun 1, 2026
3 checks passed
@arseny-dragonflydb arseny-dragonflydb deleted the docs/acl-rules-location-update branch June 1, 2026 09:52
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