Skip to content

Conversation

@paulOsinski
Copy link
Contributor

This introduces a few changes to our documentation

  • theming by version (os vs pro)
  • new top navigation bar
  • new organization overall

Copy link
Contributor

@mtesauro mtesauro left a comment

Choose a reason for hiding this comment

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

Approved

Copy link
Member

@valentijnscholten valentijnscholten left a comment

Choose a reason for hiding this comment

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

@paulOsinski Some questions which are hard to answe myself as there are so many changes in the PR:

  • Does this include all changes made to the docs recently in bugfix and dev branches? I notice this PR is based against master (and GH says it's out of date with master).
  • Does this result in URL changes which will lead to users getting 404s when arriving via earlier published links to upgrade notes or via google?

@github-actions
Copy link
Contributor

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

@paulOsinski paulOsinski changed the base branch from master to bugfix January 26, 2026 23:33
@paulOsinski paulOsinski marked this pull request as draft January 26, 2026 23:34
@paulOsinski
Copy link
Contributor Author

@valentijnscholten good point regarding links. I have changed it to be based against bugfix.

I have a tool that will check for internal 404s and I'll make sure I run that before reopening the PR. When this site is deployed it will include a sitemap which should hopefully eliminate 404s from Google, but I can't say the same for any other links - there is no redirect handling

@github-actions
Copy link
Contributor

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

@paulOsinski paulOsinski marked this pull request as ready for review January 28, 2026 22:45
@paulOsinski
Copy link
Contributor Author

@valentijnscholten links are all fixed!

Copy link
Member

@valentijnscholten valentijnscholten left a comment

Choose a reason for hiding this comment

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

Not sure if I can test this so approving assuming we will fix if anything is broken after merging.

@Maffooch Maffooch requested a review from Jino-T January 30, 2026 06:14
@github-actions
Copy link
Contributor

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

@github-actions
Copy link
Contributor

github-actions bot commented Feb 2, 2026

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

@Maffooch Maffooch merged commit 7318ca2 into DefectDojo:bugfix Feb 2, 2026
91 checks passed
@paulOsinski paulOsinski mentioned this pull request Feb 2, 2026
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.

6 participants