Skip to content

Fix missing headings everywhere - #270

Merged
eldadfux merged 2 commits into
mainfrom
fix-security-no-toc
Oct 23, 2023
Merged

Fix missing headings everywhere#270
eldadfux merged 2 commits into
mainfrom
fix-security-no-toc

Conversation

@gewenyu99

@gewenyu99gewenyu99 commented Oct 22, 2023

Copy link
Copy Markdown
Contributor

What does this PR do?

Fixes missing headings ToC tags in all pages.

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

@vercel

vercelBot commented Oct 22, 2023•
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

NameStatusPreviewCommentsUpdated (UTC)
website✅ Ready (Inspect)Visit Preview💬 Add feedbackOct 23, 2023 7:31pm

@gewenyu99
gewenyu99 requested review from TorstenDittmann and eldadfux and removed request for TorstenDittmannOctober 22, 2023 18:58
eldadfux
eldadfux reviewed Oct 22, 2023
View reviewed changes
Comment threadsrc/routes/docs/apis/realtime/+page.markdoc Outdated
The examples below will show you how you can use Realtime in various ways.

## Subscribe to a Channel
## Subscribe to a Channel {% #subscribe-to-one-channel %}

@eldadfuxeldadfuxOct 22, 2023

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we keep the hashes 1:1 with titles? we're not consistent in many places.

gewenyu99
gewenyu99 commented Oct 23, 2023
View reviewed changes
Comment threadsrc/routes/docs/advanced/platform/permissions/+page.markdoc Outdated
Comment threadsrc/routes/docs/advanced/platform/permissions/+page.markdoc Outdated
Comment threadsrc/routes/docs/advanced/self-hosting/tls-certificates/+page.markdoc Outdated
Comment threadsrc/routes/docs/apis/realtime/+page.markdoc Outdated
Comment threadsrc/routes/docs/apis/realtime/+page.markdoc Outdated
Comment threadsrc/routes/docs/products/databases/relationships/+page.markdoc Outdated
Comment threadsrc/routes/docs/products/storage/upload-download/+page.markdoc Outdated
Comment threadsrc/routes/docs/products/storage/upload-download/+page.markdoc Outdated
Comment threadsrc/routes/docs/tutorials/react/step-6/+page.markdoc Outdated
Comment threadsrc/routes/docs/tutorials/sveltekit/step-6/+page.markdoc Outdated
@eldadfux
eldadfux merged commit b179f0a into mainOct 23, 2023
This was referenced Nov 1, 2023
Closed
Closed
@gewenyu99
gewenyu99 deleted the fix-security-no-toc branch November 21, 2023 19:19
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@eldadfuxeldadfuxeldadfux approved these changes
@TorstenDittmannTorstenDittmannAwaiting requested review from TorstenDittmann

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@gewenyu99@eldadfux