Skip to content

Upstream/Downstream Connectors docs overhaul - #15266

Open
paulOsinski wants to merge 4 commits into
DefectDojo:bugfixfrom
paulOsinski:connections-docs-overhaul
Open

Upstream/Downstream Connectors docs overhaul#15266
paulOsinski wants to merge 4 commits into
DefectDojo:bugfixfrom
paulOsinski:connections-docs-overhaul

Conversation

@paulOsinski

@paulOsinski paulOsinski commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

This PR catches our documentation up with Upstream/Downstream Connectors, a merge of our Integrators and (upstream only) Connectors fields. There's a lot of content here so I'll break it down

  • updates to doc organizations: Connectors is now its own section which includes both Upstream and Downstream
  • addition of new Connectors button to header nav
  • updates to doc navigation and aliases to prevent 404 errors
  • refreshes screenshots
  • moves Jira content to Connectors section (OS and Pro)
  • aligns terminology within docs

out of scope but also included

  • switched external pointing links to internal ones so that docs users in dev don't get taken to our production site

@accesslint accesslint Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Found 10 issues across 5 rules (4 WCAG, 6 Best Practice).

Comment thread docs/assets/js/custom.js
const storedVersion = localStorage.getItem("version") || "opensource";
console.log("[VersionToggle] Stored version:", storedVersion);
setVersion(storedVersion);
// custom-head.html already stamped the stored edition on <html> before

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

WCAG 2.4.2: Document is missing a <title> element.

Documents must have a <title> element to provide users with an overview of content.

Details

Page text: "// custom js // Mobile navigation: the site sets doks.bootstrapJavascript = false, so no // Bootstrap plugins load and the header's offcanvas toggles "

Screen reader users rely on page titles to identify and navigate between tabs/windows. Add a descriptive <title> element in <head> that summarizes the page purpose. Keep titles unique across the site, placing specific content before the site name (e.g., 'Contact Us - Acme Corp').


Best Practice: Page has no mechanism to bypass repeated content. Add a <main> landmark or skip link.

Page must have a mechanism to bypass repeated blocks of content.

Details

Missing: no landmarks (<main>, <nav>, <header>, <footer>), no skip link, no headings

Keyboard users must be able to skip repetitive content like navigation. Provide a skip link at the top of the page that links to the main content (e.g., <a href="#main">Skip to main content</a>), or use a <main> landmark. Screen readers can jump directly to landmarks, so a properly marked-up <main> element satisfies this requirement.


Best Practice: Page does not contain a level-one heading.

Page should contain a level-one heading.

Details

A level-one heading (<h1> or role='heading' with aria-level='1') helps users understand the page topic and provides a landmark for screen reader navigation. Each page should have exactly one h1 that describes the main content, typically matching or similar to the page title.


Best Practice: Page has no main landmark.

Page should have exactly one main landmark.

Details

The main landmark contains the primary content of the page. Screen readers allow users to jump directly to main content. Use a single <main> element (or role='main') to wrap the central content, excluding headers, footers, and navigation.


WCAG 3.1.1: <html> element missing lang attribute.

The <html> element must have a lang attribute.

Details

Page text sample: "// custom js // Mobile navigation: the site sets doks.bootstrapJavascript = false, so no // Bootstrap plugins load and the header's offcanvas toggles are dead markup. // Importing the Offcanvas plugin"

Screen readers use the lang attribute to determine which language rules and pronunciation to use. Without it, content may be mispronounced. Set lang to the primary language of the page (e.g., lang='en' for English, lang='es' for Spanish).

--bs-btn-active-color: #fff;
font-weight: 500; // Medium for buttons
}
// Selected state keys off the edition stamped on <html> before first

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

WCAG 2.4.2: Document is missing a <title> element.

Documents must have a <title> element to provide users with an overview of content.

Details

Page text: "// ============================================================ // DefectDojo Docs design system // // Imported last in doks-core app.scss — everythin"

Screen reader users rely on page titles to identify and navigate between tabs/windows. Add a descriptive <title> element in <head> that summarizes the page purpose. Keep titles unique across the site, placing specific content before the site name (e.g., 'Contact Us - Acme Corp').


Best Practice: Page has no mechanism to bypass repeated content. Add a <main> landmark or skip link.

Page must have a mechanism to bypass repeated blocks of content.

Details

Missing: no landmarks (<main>, <nav>, <header>, <footer>), no skip link, no headings

Keyboard users must be able to skip repetitive content like navigation. Provide a skip link at the top of the page that links to the main content (e.g., <a href="#main">Skip to main content</a>), or use a <main> landmark. Screen readers can jump directly to landmarks, so a properly marked-up <main> element satisfies this requirement.


Best Practice: Page does not contain a level-one heading.

Page should contain a level-one heading.

Details

A level-one heading (<h1> or role='heading' with aria-level='1') helps users understand the page topic and provides a landmark for screen reader navigation. Each page should have exactly one h1 that describes the main content, typically matching or similar to the page title.


Best Practice: Page has no main landmark.

Page should have exactly one main landmark.

Details

The main landmark contains the primary content of the page. Screen readers allow users to jump directly to main content. Use a single <main> element (or role='main') to wrap the central content, excluding headers, footers, and navigation.


WCAG 3.1.1: <html> element missing lang attribute.

The <html> element must have a lang attribute.

Details

Page text sample: "// ============================================================ // DefectDojo Docs design system // // Imported last in doks-core app.scss — everything here wins the // cascade. Selectors defined here"

Screen readers use the lang attribute to determine which language rules and pronunciation to use. Without it, content may be mispronounced. Set lang to the primary language of the page (e.g., lang='en' for English, lang='es' for Spanish).

@github-actions

Copy link
Copy Markdown
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@paulOsinski
paulOsinski marked this pull request as draft July 20, 2026 14:48
@Maffooch Maffooch added this to the 3.2.0 milestone Jul 22, 2026
@Maffooch Maffooch modified the milestones: 3.2.0, 3.2.100 Jul 31, 2026

@accesslint accesslint Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👏 All fixed. Great work.

@github-actions

Copy link
Copy Markdown
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

paulOsinski and others added 2 commits July 31, 2026 16:22
Introduce a single top-level Connectors section that merges the former
API Connectors (upstream) and Integrations (downstream) docs, adopting
"Connectors" as the umbrella term across the docs.

- new docs/content/connectors/ section: about, upstream/, downstream/,
  os_jira/, and issue-tracking overview, moved from import_data/pro/connectors
  and issue_tracking/
- single "Connectors" top-nav item and homepage card; sectionNav updated
- PagerDuty, Linear, and ServiceNow SecOps added to the downstream reference;
  connector reference deduplicated
- align terminology across glossary, feature lists, comparison tables, and
  asset-modelling pages; add redirect aliases for the old paths

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Replace absolute https://docs.defectdojo.com/... links with root-relative
/... paths across parser reference pages and a few other docs, so links
resolve within whichever environment (dev/staging/prod) is serving the
docs instead of always jumping to the production site. Also fixes a few
dead self-links.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@paulOsinski
paulOsinski force-pushed the connections-docs-overhaul branch from a438cec to 2ec927c Compare July 31, 2026 20:23
@paulOsinski
paulOsinski marked this pull request as ready for review July 31, 2026 20:24
@paulOsinski paulOsinski changed the title Connections docs overhaul Upstream/Downstream Connectors docs overhaul Jul 31, 2026
Restore add_edit_connectors{,_2,_3}.png and integrators_3.png to their
base (bugfix) versions. The refreshed screenshots are no longer part of
this change; the Connectors docs keep the existing images.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants