Skip to content

[improve][doc] improve doc and fix errors - #18860

Merged
tisonkun merged 5 commits into
apache:masterfrom
labuladong:improve-connector-doc
Dec 13, 2022
Merged

[improve][doc] improve doc and fix errors#18860
tisonkun merged 5 commits into
apache:masterfrom
labuladong:improve-connector-doc

Conversation

@labuladong

@labuladong labuladong commented Dec 10, 2022

Copy link
Copy Markdown
Contributor

Modifications

Most modification is style improvement and obvious error fix.

The notable changes are in debug function with localrun mode:

image

Add -original and protobuf dependency. If not, the localrun function cannot start.

Documentation

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

@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 Dec 10, 2022
@tisonkun tisonkun changed the title [Improve][doc] improve doc and fix errors [improve][doc] improve doc and fix errors Dec 10, 2022
Comment thread site2/docs/deploy-bare-metal.md Outdated
Comment thread site2/docs/deploy-bare-metal.md Outdated
Comment thread site2/docs/functions-debug-localrun.md Outdated
Comment on lines 1 to 5
---
Id: how-to-landing
title: How-to
sidebar_label: “”
title: How to landing
sidebar_label: "How to landing"
---

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.

I think this is a landing page that intends to use this title. Please revert these changes.

cc @momo-jun @DaveDuggins

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yes, it's a landing page for a serial of how-to topics.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Does sidebar_label: “” need to revert?

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.

@labuladong you can change it to sidebar_label: "" but actually Docu recognizes both " and .

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

😂 Seems sidebar_label must be “”. if I change it to "", error occurs:

image

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's a very tricky way to avoid this error. I'm not sure of the reason, I'm reverting it for now.

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.

I believe you can remove the sidebar_label at all. Since these pages don't show in the sidebar.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I suggest keeping this field. I guess it's very likely that they will appear in the sidebar soon or later :)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I've tested, remove sidebar_label: “” works fine. I think removing it is better because using“” is too tricky and not a good practice in the document.

they will appear in the sidebar soon or later.

We can just add it at any time if we need it.

Comment thread site2/docs/io-quickstart.md Outdated
Comment thread site2/docs/concepts-multiple-advertised-listeners.md Outdated
@labuladong
labuladong force-pushed the improve-connector-doc branch from a587d42 to a6e3c80 Compare December 13, 2022 02:44

@momo-jun momo-jun left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice fix!

@labuladong
labuladong requested a review from tisonkun December 13, 2022 03:23

@tisonkun tisonkun left a comment

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.

Thanks for your contribution @labuladong!

Merging...

@tisonkun
tisonkun merged commit 4a0b775 into apache:master Dec 13, 2022
@labuladong
labuladong deleted the improve-connector-doc branch December 13, 2022 04:21
Demogorgon314 pushed a commit to Demogorgon314/pulsar that referenced this pull request Dec 26, 2022
Demogorgon314 pushed a commit to Demogorgon314/pulsar that referenced this pull request Dec 29, 2022
lifepuzzlefun pushed a commit to lifepuzzlefun/pulsar that referenced this pull request Jan 10, 2023
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.

3 participants