BP-2432: Extension management tasks#222
BP-2432: Extension management tasks#222jeff-matthews wants to merge 4 commits intoBP-2432-extension-mgmtfrom
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
WalkthroughDocumentation reorganized to an extension-centric OpenGraph management workflow. Added explicit sections for Extensions, Before you begin, Workflow, Register/Delete/Update extension, Upload data, and Explore and analyze; visuals and a concept glossary were introduced or revised. (48 words) Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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 |
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/opengraph/extensions/manage.mdx`:
- Line 63: Fix the malformed HTML break tag in the Mermaid node label: locate
the node string f["Register</br/>extension schema"] and replace the incorrect
"</br/>" with the correct "<br/>" so it reads f["Register<br/>extension schema"]
to match the other diagram nodes.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 866450af-51c1-41ff-86a9-aee37dc9afb4
⛔ Files ignored due to path filters (2)
docs/images/opengraph/extensions/manage/active-extensions-list.pngis excluded by!**/*.pngdocs/images/opengraph/extensions/manage/upload-extension-schema.pngis excluded by!**/*.png
📒 Files selected for processing (1)
docs/opengraph/extensions/manage.mdx
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/opengraph/extensions/manage.mdx`:
- Line 149: Update the sentence starting "To update an extension, upload the new
version using the same process as registering a new extension. BloodHound
validates the new schema and replace the old version with the new one." by
changing the verb "replace" to "replaces" so the subject-verb agreement is
correct: "BloodHound validates the new schema and replaces the old version with
the new one."
- Line 161: The wording "Click the **Upload File** modal" is misleading; update
the sentence that begins with "Click the **Upload File** modal" so it instructs
users to click the button (or control) that opens the modal — e.g., change to
"Click the **Upload File** button to open a file system dialog, or drag and drop
the data payload file(s) onto the canvas." Ensure the revised text replaces the
exact phrase "Click the **Upload File** modal" in the manage.mdx content.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 9f95c718-e8f0-46df-a85f-5e374b482a15
📒 Files selected for processing (1)
docs/opengraph/extensions/manage.mdx
1 participant
Purpose
This pull request (PR) supplements the extensions management concepts in #208 with task-based details.
Staging
https://specterops-bp-2432-extension-mgmt-tasks.mintlify.app/opengraph/extensions/manage#manage-extensions
Summary by CodeRabbit