Repository navigation
docs: list loki and clickhouse in the header adapters menu - #885
Merged
Merged
Conversation
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
3 Skipped Deployments
|
Contributor
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Contributor
|
Thank you for following the naming conventions! 🙏 |
HugoRCD
enabled auto-merge (squash)
October 17, 2026 11:21
@evlog/cli
evlog
@evlog/nuxthub
@evlog/signals
@evlog/telemetry
commit: |
This branch was successfully deployed
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
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.
Refs EVL-675.
What changed
The docs header Adapters menu (
apps/docs/app/components/app/AppHeaderCenter.vue) listed 10 drain adapters. Grafana Loki and ClickHouse are added after Datadog, with the Simple Iconsgrafanaandclickhouseicons and links to their hybrid pages. The menu now links all 12 drain adapters, matching the landing count.This is the only file changed in this PR. The other items of the EVL-675 follow-up were already on
mainthrough #882 (cf3a43b): the Sentry status order in03.sentry.md, the Loki and ClickHouse rows inpackages/evlog/README.md, theevlog initcatalog entries, the telemetry keys, the tests, the changeset, and the landing em dashes. Nothing else is touched here, andbench/is not included.Rendered check (count 12)
apps/docs.output, served on localhost) renders the Adapters dropdown open, with the Grafana Loki and ClickHouse links in the nav. The adapter links in the dropdown are the 12 drain adapters: Axiom, PostHog, Sentry, Better Stack, Datadog, Sazabi, Grafana Loki, ClickHouse, OTLP, HyperDX, File System, NuxtHub. A screenshot was captured locally; the Vercel preview screenshot is not yet attached (see below).dpl_BhFu3MCD9XvUuqGmCgFXsH5D1TwC(evlog-docs-e41mquvb3-hrcd.vercel.app) was stillBUILDINGabout 12 minutes after the push. It has not been checked yet. A preview screenshot will be added when it is READY.LandingStats.vueshows12drain adapters,landing.md:242reads "12 drain adapters, plug and play", andlanding.md:303reads "twelve drain adapters". Checked on the local dev server against the same content, not on a Vercel preview (the Vercel connection is bound to the docs project).Verification
pnpm run lintat the repo root: 25 of 25 tasks successful.apps/docspnpm run build: exit 0 (cold, about 2 minutes).The script fails on duplicates. Its negative control on an earlier run (a scratch copy with a duplicated Loki row) exited 1 with
FAIL duplicate adapter row. That run also printed FAIL lines for files missing from the scratch copy, which is noise from the copy and not from the check.No changeset:
apps/docsis not a published package.ELI5
The docs site has a menu of the tools evlog can send logs to. It listed 10 of the 12, and the homepage says 12. This adds the last two to the menu, so the count matches.