-
Notifications
You must be signed in to change notification settings - Fork 34
Added missing nav cards #890
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
Conversation
📝 WalkthroughWalkthroughNavigation structure expanded with 35+ new card entries (ids 5–40) including sections for Atto's Home, AI Agents, Auto Healing, Accessibility Testing, SAP/Desktop/Windows Automation, and GenAI Capabilities. Existing entries renamed and reordered. A new top-level "byok" (Bring Your Own Key) navigation entry added to left-nav-title.json pointing to GenAI Capabilities documentation. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested reviewers
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this 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: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
src/components/navcards.jsxsrc/left-nav-title.json
⏰ 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/components/navcards.jsx (5)
31-48: LGTM!The new navigation cards for "Atto's Home", "AI Agents", and "Auto Healing" are well-structured with clear titles, descriptive content, and consistent path patterns.
55-84: LGTM!The renumbered and updated entries (Test Data Management, Elements (Objects), Test Step Recorder, Test Suites, Test Plans, Test Runs/Executions) are consistent. The title updates improve clarity.
181-246: LGTM!The new navigation cards for Accessibility Testing, SAP Automation, Desktop Automation, Windows Automation variants, GenAI Capabilities, Testsigma Tunnel, Activity Monitoring, and Debugging are well-structured with consistent formatting, unique IDs, and descriptive content.
85-90: LGTM!The "Live Editor" card with the updated path and description is properly structured.
91-180: LGTM!The renumbered entries (ids 15-29) maintain their original content with updated sequential IDs. The changes integrate well with the new navigation card structure.
| "end-to-end-testing": {"/docs/test-plans/end-to-end-testing/": "End-to-End Testing"}, | ||
| "byok": {"/docs/genai-capabilities/byok/": "Bring Your Own Key (BYOK)" } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Duplicate byok key in JSON.
The "byok" key on line 764 duplicates entries already present on lines 695 and 717. In JSON, duplicate keys result in the last value overriding previous ones. Since line 764's path (/docs/genai-capabilities/byok/) is identical to line 695, this addition appears redundant.
If this is intentional to override line 717's different path (/docs/atto/generative-ai/byok/), consider removing or consolidating the duplicate entries to avoid confusion.
🤖 Prompt for AI Agents
In src/left-nav-title.json around lines 763 to 764 there is a duplicate "byok"
key (also present at lines 695 and 717) which causes last-value override; remove
the redundant entry at line 764 (or consolidate by keeping only the single
intended mapping for "byok" and deleting the other duplicate(s)), ensuring each
JSON key is unique and the correct path (either /docs/genai-capabilities/byok/
or /docs/atto/generative-ai/byok/) remains.
Added missing nav cards in Docs Home

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