docs: add Deploy to Zopday buttons and repair the dead chart links - #319
Merged
Merged
Conversation
Every chart link in the top-level README pointed at https://helm.zop.dev/<chart>, and all 24 of them return 404. The live page is the chart browser at https://helm.zop.dev/src/readme.html?id=<chart>, so every link now uses that form. Each one was checked against the running site rather than assumed. Adds the Deploy to Zopday button to the seven application charts' READMEs, each pointing at https://zop.dev/zopday/app/deploy?install=<chart>, and a Deploy column to the applications table so the same button is reachable from the top-level README. Datasource charts deliberately do not get one - they are not things a user deploys as an app. Eight published charts were missing from the tables entirely: scylladb under datasources, and openobserve-standalone, uptime-monitoring, zookeeper, zookeeper-operator, solr-operator and karpenter-gcp under others. uptime-monitoring was the one published chart with no README at all, so linking it would have landed on the site's not-found state. Written from its templates: what it probes, the two queries it exists for (probe_success and certificate expiry), its values, and the fact that its Blackbox objects are not release-prefixed so it can only be installed once per namespace. Also repairs two dead links in the holmesgpt README - holmesgpt.dev moved its docs under a /latest/ prefix. Every URL in every README now resolves: 45 in the top-level README, 27 across the chart READMEs. The two exceptions are intentional - oauth2.googleapis.com is a POST-only endpoint that 404s on GET.
The centred block sat under the heading and pushed the description down. A right-floated anchor placed before the H1 puts it on the title line instead, flush right. Verified against the live stylesheet rather than assumed: docs/src/css/readme.css sets #readme-content img to display:block, which does not defeat a float. Rendered geometry measured at float:right, 169x36, overlapping the h1 box. The anchor stays outside the heading so readme.js, which labels its sidebar from heading.textContent, still reads a clean title.
jatintalgotra-zd
approved these changes
Aug 19, 2026
7 tasks done
arunesh-j
added a commit
that referenced
this pull request
Aug 28, 2026
docs/index.yaml conflicted again because both sides regenerated it wholesale - main gained the qdrant chart (#310) and the README fixes (#319). Resolved the same way: discard both versions and re-run helm repo index . --url https://helm.zop.dev over the merged docs/ directory. Verified the only semantic change against main is the added n8n v0.0.1 entry - 184 tarball digests match, nothing removed, nothing altered, and qdrant v0.0.1 from main is intact. Also adds n8n to the applications table in the top-level README. #319 landed that table with a Deploy column while this branch was open, so without this the chart would merge unlisted. Both of its links were checked live.
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
2 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.
The links were all dead
Every chart link in the top-level README pointed at
https://helm.zop.dev/<chart>. All 24 of them return 404 — that path was never served. The live page is the chart browser:Every link now uses the working form, and each one was checked against the running site rather than assumed.
Deploy to Zopday buttons
Added to all seven application charts' READMEs, directly under the H1:
holmesgpt,jupyterhub,litellm,localai,outline,superset,wordpress— each with its owninstall=<chart>. The top-level applications table also gains a Deploy column carrying the same button, so it is reachable without opening a chart first.Datasource charts deliberately do not get one: they are not something a user deploys as an app.
n8ngets the same button on #311 rather than here, so it cannot land without it.Eight charts were missing from the tables
All published, none listed:
scylladb(datasources) andopenobserve-standalone,uptime-monitoring,zookeeper,zookeeper-operator,solr-operator,karpenter-gcp(others).uptime-monitoring had no README
It was the one published chart with none, so the link I was adding would have landed on the site's not-found illustration. Written from its templates: what it probes, the two queries it exists for (
probe_success, and certificate expiry via(probe_ssl_earliest_cert_expiry - time()) / 86400), its values, and one thing worth knowing before installing — its Blackbox objects (blackbox-exporter,blackbox-config,prometheus-config) are not release-prefixed, so two releases in one namespace collide. Documented rather than changed, since renaming them is a breaking change for anyone already running it.Also
Two dead links in the holmesgpt README:
holmesgpt.devmoved its docs under a/latest/prefix, so/ai-providers/and/data-sources/both 404. Repaired and verified.Verification
Every URL in every README resolves — 45 in the top-level README, 27 across the chart READMEs, checked live:
readme.html?id=chart links → 200zop.dev/zopday/app/deploy?install=links → 200One deliberate exception:
https://oauth2.googleapis.com/tokenin a chart README is a POST-only endpoint and correctly 404s on GET.