docs: split Attaching Files into OS and Pro guides with screenshots#15104
Open
Maffooch wants to merge 1 commit into
Open
docs: split Attaching Files into OS and Pro guides with screenshots#15104Maffooch wants to merge 1 commit into
Maffooch wants to merge 1 commit into
Conversation
Replace the outdated, Findings-only add_files.md (which incorrectly claimed image-only support) with an audience-scoped OS/Pro pair, following the OS__/PRO__ convention already used by the risk_acceptance docs. - OS__add_files.md: Manage Files workflow for Findings, Engagements, and Tests; corrected the supported file-type list to match DD_FILE_UPLOAD_TYPES; added fresh screenshots. - PRO__add_files.md: documents the Pro UI flow (gear > Add File, the Files tab, and the row menu's Edit File Name / Delete File). - Add OS_/PRO_ prefixed screenshots; preserve the old /add_files/ URL via an alias on the OS page. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Maffooch
force-pushed
the
docs/attach-files-os-pro
branch
from
94abd3f to
36aa8f1
Compare
paulOsinski
approved these changes
Jun 30, 2026
valentijnscholten
approved these changes
Jun 30, 2026
Jino-T
approved these changes
Jul 1, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Replaces the outdated, Findings-only
add_files.mdwith an audience-scoped OS/Pro pair documenting the file-attachment ("uploaded files") feature, following theOS__/PRO__convention used by therisk_acceptancedocs in the same folder.The old page only covered Findings and incorrectly claimed image-only support. The feature actually attaches files to Findings, Engagements, and Tests and accepts the full
DD_FILE_UPLOAD_TYPESset (.pdf,.csv,.zip,.xlsx, …), not just images.Changes
OS__add_files.md(audience: opensource) — Manage Files workflow for Findings, Engagements, and Tests; corrected supported file types; new modern-theme screenshots (action menu, upload form, Files panel with thumbnail).PRO__add_files.md(audience: pro) — Pro UI flow: gear (⚙) → Add File, the Files tab, and the row (⋮) menu's Edit File Name / Delete File.OS_/PRO_prefixed screenshots; removed the now-deletedadd_files.md./add_files/URL via an alias on the OS page (redirects to/os__add_files/).Verification
hugo --minify --gcbuilds clean; both/os__add_files/and/pro__add_files/render with all images; the/add_files/alias redirects correctly.dojo/settings/settings.dist.py:236.🤖 Generated with Claude Code
Shortcut: sc-13299