fix(api): point the Activity Log sidebar entry at its real page - #12396
Merged
mergify[bot] merged 1 commit intoAug 17, 2026
Conversation
The OpenAPI spec renamed the `eventlogs` tag to `activity_log` in a schema sync, following the product rename of Event Logs to the Activity Log. Every `/api/*` page is generated from those tags by `src/pages/api/[tag].astro`, so the rename deleted `/api/eventlogs` and created `/api/activity-log`. The sidebar entry was never updated, and it renders on every page of the site: a 404 reachable in one click from 225 pages. Point the entry at `/api/activity-log` and rename it to match the product. Redirect the old URL, which has been live long enough to be indexed and bookmarked. `TAG_LABELS` / `TAG_DESCRIPTIONS` are keyed by spec tag, so the rename also orphaned the curated copy for this section — the generated page silently fell back to `humanizeTag` plus a generated sentence. Rekey both entries to `activity_log` rather than deleting them, and widen the description: the Activity Log is a timeline of every Mergify event for a repository, not just pull request activity. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Change-Id: I400be77552128bab6a8965b4b8bb5dae322f6802
Member
Author
|
This pull request is part of a Mergify stack:
|
Contributor
Merge Protections🟢 All 6 merge protections satisfied — ready to merge. Show 6 satisfied protections🟢 🤖 Continuous Integration
🟢 👀 Review Requirements
🟢 Enforce conventional commitMake sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
🟢 🔎 Reviews
🟢 📕 PR description
🟢 🚦 Auto-queueWhen all merge protections are satisfied, this pull request will be queued automatically. |
There was a problem hiding this comment.
Pull request overview
Updates the docs site’s API Reference navigation and curated tag metadata to match the OpenAPI tag rename from eventlogs to activity_log, ensuring the “Activity Log” API page is reachable from the global sidebar and preserving the old URL via redirects.
Changes:
- Update the API sidebar entry from “Event Logs” (
/api/eventlogs) to “Activity Log” (/api/activity-log). - Rekey curated OpenAPI tag label/description from
eventlogstoactivity_logand refine the description copy. - Add redirects from
/api/eventlogs(with and without trailing slash) to the new Activity Log page.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/content/navItems.tsx | Updates the API sidebar entry label and path to the new Activity Log page. |
| src/components/ApiReference/openapi.ts | Rekeys and updates curated tag labels/descriptions to match the renamed OpenAPI tag. |
| public/_redirects | Adds redirects to preserve the old /api/eventlogs URL. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
JulianMaurin
approved these changes
Aug 17, 2026
jd
approved these changes
Aug 17, 2026
Contributor
Merge Queue Status
This pull request spent 2 minutes 51 seconds in the queue, including 2 minutes 8 seconds running CI. Required conditions to merge
|
60 tasks
mergify
Bot
deleted the
devs/sileht/api-eventlogs-404/point-activity-log-sidebar-entry-real-page--400be775
branch
August 17, 2026 08:53
62 tasks
mergify Bot
pushed a commit
that referenced
this pull request
Aug 17, 2026
`check:links` passed `--skip 'https?://'` to drop off-site links. linkinator applies skip rules to the absolute URL of everything it fetches, and when it is pointed at a directory it serves the build over HTTP and crawls it — so that unanchored rule matched its own crawl root. The command exited 0 having scanned zero links, and had been doing so on every pull request: the sidebar link to the deleted `/api/eventlogs` page shipped through a green link check that never requested a single URL. Anchor the rule and exclude the local server. Both of its hostnames have to be excluded, not just one: linkinator binds to `127.0.0.1` but builds its trailing-slash redirects against `localhost`, so covering only the bind address still skips nearly every page at the redirect hop. The check now crawls 1355 links. It finds no pre-existing breakage — with the previous commit in place the site is clean — and it fails on a reintroduced `/api/eventlogs`. Three supporting changes: - Move the settings into `linkinator.config.mjs`. The `.json` linkinator loads by default was already dead (CLI flags replace config wholesale, so its `skip` list, `^/api` included, never applied). `.mjs` also lets the rule carry the comment explaining the trap and be asserted in a unit test — the failure mode here is a check that silently stops checking, which no amount of green CI would reveal. - Crawl `/enterprise/` as a second entry point. Nothing outside that section links into it — it renders its own nav tree and is filtered out of the sitemap — so recursing from `/` alone reaches none of its eight pages and a broken link there would pass forever. - Lower the concurrency and enable `retryErrors`. Now that pages are actually fetched, linkinator's own static server drops an occasional connection, surfacing as a status-0 failure on a different file each run. `retryErrors` covers status 0, 5xx and 429 only — a 404 is never retried, so this costs nothing in strictness. Its count and jitter are left on the built-in defaults on purpose: meow declares defaults for both, so a value set in the config file would be silently overridden — the same class of dead config as the `.json` this replaces. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Depends-On: #12396
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.
The OpenAPI spec renamed the
eventlogstag toactivity_login a schemasync, following the product rename of Event Logs to the Activity Log. Every
/api/*page is generated from those tags bysrc/pages/api/[tag].astro, sothe rename deleted
/api/eventlogsand created/api/activity-log. Thesidebar entry was never updated, and it renders on every page of the site: a
404 reachable in one click from 225 pages.
Point the entry at
/api/activity-logand rename it to match the product.Redirect the old URL, which has been live long enough to be indexed and
bookmarked.
TAG_LABELS/TAG_DESCRIPTIONSare keyed by spec tag, so the rename alsoorphaned the curated copy for this section — the generated page silently fell
back to
humanizeTagplus a generated sentence. Rekey both entries toactivity_lograther than deleting them, and widen the description: theActivity Log is a timeline of every Mergify event for a repository, not just
pull request activity.
Co-Authored-By: Claude Opus 5 (1M context) noreply@anthropic.com