Skip to content

[fix][doc] Workaround Docusaurus links issue - #18485

Merged
tisonkun merged 3 commits into
apache:masterfrom
tisonkun:site-293
Nov 16, 2022
Merged

[fix][doc] Workaround Docusaurus links issue#18485
tisonkun merged 3 commits into
apache:masterfrom
tisonkun:site-293

Conversation

@tisonkun

Copy link
Copy Markdown
Member

This is the counterpart of apache/pulsar-site#293.

It's a bit clumsy to update multiple occurrences for the same purpose :/

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository:

Signed-off-by: tison <wander4096@gmail.com>
@github-actions github-actions Bot added the doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. label Nov 15, 2022
@tisonkun

Copy link
Copy Markdown
Member Author

Perhaps usingpathname:// to simplify code a bit. https://docusaurus.io/docs/markdown-features/assets#static-assets

Pending for improving...

Signed-off-by: tison <wander4096@gmail.com>
Signed-off-by: tison <wander4096@gmail.com>
@tisonkun
tisonkun merged commit 9da5980 into apache:master Nov 16, 2022
@tisonkun

Copy link
Copy Markdown
Member Author

After a discussion with @momo-jun, I'm trying to add a version dropdown list in the C++ API documents so that we can always link to the latest version.

References:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Your PR contains doc changes, no matter whether the changes are in markdown or code files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants