Skip to content

Move API content to the new toplevel API section, link the TopNav and light it up in the search - #3439

Merged
borland merged 9 commits into
mainfrom
orion/move-api
Sep 2, 2026
Merged

Move API content to the new toplevel API section, link the TopNav and light it up in the search#3439
borland merged 9 commits into
mainfrom
orion/move-api

Conversation

@borland

@borland borland commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Our new UI design intends for all the API-related docs to go under /docs/api, and be accessible via an "API" button in the top-level nav bar. The API docs also include generated docs, synced from the Octopus Server repository on a regular basis.

These generated docs were already present but not linked anywhere. This PR finishes the job, as follows

  • The top-nav "API" button now goes to /docs/api

    • The API subsection of the site includes the generated docs, and has it's own special navigation menu which is aware of the API structure, with different UI layouts.
  • Pages that were previously inside the /docs/octopus-rest-api folder have now moved to /docs/api

  • Fixed all the links from /docs/octopus-rest-api/** to now go to /docs/api/**

  • Redirects have been installed for /docs/octopus-rest-api so external links pointing to the old URL's should still work

  • The "under construction" code has been removed, so the pagefind search index should now include generated API docs. Including change to the "Facet" thing

Reducing Risk

I've tested locally, but will re-test properly on the staging site (will check off as I've done)

  • TopNav goes to api
  • TopNav API section and other sections hilight appropriately
  • Scroll-to-visible fix in the nav works across all sections (shaun recently implemented this, just checking I haven't re-broken it)
  • Generated content from the API docs appears in the search

Hit some old octopus-rest-api/ links and check they redirect to the correct new URL

  • docs/octopus-rest-api/octopus.client
  • docs/octopus-rest-api/examples/artifacts
  • docs/octopus-rest-api/openid-connect

Redirects are annoying to test on the staging site as the redirects hardcode links to octopus.com/docs which won't be valid until this PR merges. But if you take the /docs/api/...whatever part of the URL from the redirect and check it on the staging site, it's good

Please bash at it and do whatever you think might be worth testing

@team-marketing-branch-protections

Copy link
Copy Markdown

Pull request environment is available at https://stoctodocspr3439.z22.web.core.windows.net.

You can view the ephemeral environment status in Octopus Deploy.

This environment will be automatically deprovisioned when the pull request is closed, or after 7 days of inactivity.

Comment thread .github/CODEOWNERS

# updates to the octopus rest api tentacle exe command line page are reviewed by team-executions-foundations

/docs/octopus-rest-api/tentacle.exe-command-line/index.md @OctopusDeploy/team-executions-foundations

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Did not think of this!! thanks for picking up on it

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I didn't either, just found it by accident in a global file search 😄

Nobody pays attention to these codeowners things, we could probably delete the whole file, but I took a more conservative approach

Comment thread package.json Outdated
"dev:watch": "onchange 'src/**/*.{js,mjs,ts,astro,css}' -- prettier --write --plugin=prettier-plugin-astro {{changed}}",
"astro": "astro",
"build": "astro build",
"build": "node --max-old-space-size=8192 node_modules/astro/bin/astro.mjs build",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@borland Are these intended? looks dodge

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

They were intended (my machine kept crashing because the default nodejs 4gb) but on reflection it might be good to take them out.

@enf0rc3 enf0rc3 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, i tested and confirmed the new url nav menu and also search that is filtering correctly.

There were some things on code-owners worth double checking as i think we moved some things.

Comment thread .github/CODEOWNERS Outdated
Comment thread .github/CODEOWNERS Outdated
Comment thread .github/CODEOWNERS Outdated
Comment thread package.json Outdated
Comment thread package.json Outdated
Co-authored-by: William Laugesen <will.laugesen@gmail.com>
Co-authored-by: Orion Edwards <orion.edwards@gmail.com>
@borland
borland enabled auto-merge (squash) September 2, 2026 01:10
@borland
borland merged commit d6c6dfe into main Sep 2, 2026
7 checks passed
@borland
borland deleted the orion/move-api branch September 2, 2026 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants