Skip to content

fix(batch-changes): repair broken permissions table#1770

Open
eugenio-sanchez-sg wants to merge 1 commit intomainfrom
fix/batch-changes-permissions-table
Open

fix(batch-changes): repair broken permissions table#1770
eugenio-sanchez-sg wants to merge 1 commit intomainfrom
fix/batch-changes-permissions-table

Conversation

@eugenio-sanchez-sg
Copy link
Copy Markdown
Contributor

What

Fixes the broken table under "Batch Change access for each permission level" on
/batch-changes/permissions-in-batch-changes. Previously the table rendered as a
single paragraph with literal | separators visible to the reader.

Currently at Permissions in Batch Changes

You can see the page here

image

After (locally)

image

Why

The markdown was malformed in two ways:

  1. There was a blank line between the header row and the |---| separator row.
    GFM requires the separator to be on the line immediately after the header,
    otherwise the whole block falls back to a paragraph.
  2. The rows were missing leading/trailing | pipes, unlike every other table in
    the repo.

How

Rewrote the table using the standard pipe-delimited form already used elsewhere
in the docs (e.g., docs/tutorials/index.mdx). No content changes — the actions
and ✅/❌ values are identical to the previous source.

Test plan

  • pnpm run dev and visit /batch-changes/permissions-in-batch-changes
  • Confirm the "Batch Change access for each permission level" section renders
    as a real table with three columns (Batch change action / Read / Admin)
  • Confirm no other pages are affected (only this one file changed)

@vercel
Copy link
Copy Markdown

vercel Bot commented May 8, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sourcegraph-docs Ready Ready Preview, Comment May 8, 2026 7:27pm

Request Review

Copy link
Copy Markdown
Contributor

@enriquegh enriquegh left a comment

Choose a reason for hiding this comment

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

Nice! LGTM via the preview deployment

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.

2 participants