web: Fix docs links, a11y input descriptors#16671
Conversation
✅ Deploy Preview for authentik-docs canceled.
|
✅ Deploy Preview for authentik-storybook ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for authentik-integrations canceled.
|
|
|
||
| .pf-c-form__label[aria-required] .pf-c-form__label-text::after { | ||
| content: "*"; | ||
| content: "*" / "Required"; |
There was a problem hiding this comment.
TIL
https://a11ysupport.io/tests/tech__css__css_generated_content_alt
Found this while debugging links using macOS's VoiceOver
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #16671 +/- ##
==========================================
- Coverage 92.82% 92.77% -0.06%
==========================================
Files 838 838
Lines 45386 45386
==========================================
- Hits 42130 42107 -23
- Misses 3256 3279 +23
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
4d18682 to
2bfc025
Compare
|
authentik PR Installation instructions Instructions for docker-composeAdd the following block to your AUTHENTIK_IMAGE=ghcr.io/goauthentik/dev-server
AUTHENTIK_TAG=gh-877093c18b155bbed3fc8c036f52320b1cd833e0
AUTHENTIK_OUTPOSTS__CONTAINER_IMAGE_BASE=ghcr.io/goauthentik/dev-%(type)s:gh-%(build_hash)sAfterwards, run the upgrade commands from the latest release notes. Instructions for KubernetesAdd the following block to your authentik:
outposts:
container_image_base: ghcr.io/goauthentik/dev-%(type)s:gh-%(build_hash)s
global:
image:
repository: ghcr.io/goauthentik/dev-server
tag: gh-877093c18b155bbed3fc8c036f52320b1cd833e0Afterwards, run the upgrade commands from the latest release notes. |
| export function readGitBuildHash() { | ||
| try { | ||
| const commit = execSync("git rev-parse HEAD", { | ||
| encoding: "utf8", | ||
| cwd: MonoRepoRoot, | ||
| }) | ||
| .toString() | ||
| .trim(); | ||
|
|
||
| return commit; | ||
| } catch (_error) { | ||
| console.debug("Git commit could not be read."); | ||
| } | ||
|
|
||
| return process.env.GIT_BUILD_HASH || ""; | ||
| } |
There was a problem hiding this comment.
Vestigial function, removed
b915c38 to
bc91ff0
Compare
| const sampleItems: FooterLink[] = [ | ||
| { name: "authentik", href: "https://goauthentik.io" }, | ||
| { name: "authentik docs", href: "https://docs.goauthentik.io/docs/" }, | ||
| { name: "authentik docs", href: "https://docs.goauthentik.io" }, |
There was a problem hiding this comment.
Avoiding usage of import.meta.env here since the element test doesn't use the same bundle config.
| if (prerelease) { | ||
| ReleaseNotesURL = new URL("/releases", PreReleaseDocsURL); | ||
| } else { | ||
| ReleaseNotesURL = new URL(`releases/${VersionPath}`, CurrentReleaseDocsURL); | ||
|
|
||
| return [AuthentikVersion, GIT_BUILD_HASH].join("+"); | ||
| ReleaseNotesURL.hash = `fixed-in-${[major, minor, patch].join("")}`; |
There was a problem hiding this comment.
We might be able to make this anchor less fragile with a generic #latest-release-notes link in the docs.
15d903b to
f060627
Compare
f060627 to
3594984
Compare
web: Clean up constants. Fix attribute mapping. web: use previous function name. web: Fix sort. web: Use constant. web: Use prefix. web: keep using current release for notes.
66905f1 to
dedb701
Compare
dedb701 to
877093c
Compare
* main: (58 commits) web: bump the esbuild group across 2 directories with 4 updates (#16868) core, web: update translations (#16864) core: bump astral-sh/uv from 0.8.17 to 0.8.18 (#16866) website: bump @types/node from 24.5.1 to 24.5.2 in /website (#16867) web: bump @types/node from 24.5.1 to 24.5.2 in /packages/esbuild-plugin-live-reload (#16869) web: bump pino from 9.9.5 to 9.10.0 in /packages/esbuild-plugin-live-reload (#16870) web: bump @types/node from 24.5.1 to 24.5.2 in /packages/prettier-config (#16871) web: bump @types/node from 22.15.19 to 24.5.2 in /web (#16872) web: bump dompurify from 3.2.6 to 3.2.7 in /web (#16873) web: bump pino from 9.9.5 to 9.10.0 in /web (#16874) web: bump vite from 7.1.5 to 7.1.6 in /web (#16875) web: bump chromedriver from 140.0.2 to 140.0.3 in /web (#16876) lifecycle/aws: bump aws-cdk from 2.1029.1 to 2.1029.2 in /lifecycle/aws (#16877) web: Fix docs links, a11y input descriptors (#16671) website: bump the eslint group in /website with 3 updates (#16788) website: bump the build group in /website with 3 updates (#16787) web: bump the eslint group across 2 directories with 3 updates (#16790) website/docs: extends the example to include `jwt_config` for matrix/synapse (#16860) web/a11y: Flow Search (#15876) web: bump API Client version, remove Webdriver dependencies (#16836) ...
Details
This PR fixes a collection of issues that prevent doc links from using the correct domain while developing authentik. Following up on #15774, this PR also unifies a several related behaviors surrounding URL constants.
1. Version Constants
We've gradually moved runtime constants into the build step. The fixes in this PR are focused on
web/packages/core/version/node.js, moving much of error-prone version string splitting into a parsed object.2. Documentation links
Links to the docs are now injected during build time, fixing a few sources of URL mismatches such as when...
Additionally, some doc links have depended on redirects -- They've since been updated to match their current canonical URLs.
3. Miscellaneous
*being read as asterisk instead of required