Skip to content

[mkdocs] docs: apply review comments (websockets) - #932

Merged
segfaultxavi merged 3 commits into
NemProject:new-docsfrom
zero4862:docs-revision-websockets
Jul 9, 2026
Merged

[mkdocs] docs: apply review comments (websockets)#932
segfaultxavi merged 3 commits into
NemProject:new-docsfrom
zero4862:docs-revision-websockets

Conversation

@zero4862

@zero4862 zero4862 commented Jul 7, 2026

Copy link
Copy Markdown

Address review comments from @Jaguar0625 on July 2.

Comment thread mkdocs/pages/en/devbook/reference/websockets/index.md
Comment thread mkdocs/pages/en/devbook/reference/websockets/index.md Outdated
Comment thread mkdocs/pages/en/devbook/websockets/listen-transaction-flow.md
Comment thread mkdocs/pages/en/devbook/reference/websockets/index.md
Comment thread mkdocs/pages/en/devbook/reference/websockets/index.md
Comment thread mkdocs/pages/en/devbook/reference/websockets/index.md
Comment thread mkdocs/pages/en/devbook/reference/websockets/index.md
@zero4862
zero4862 force-pushed the docs-revision-websockets branch 2 times, most recently from bbeec0e to 85c8f21 Compare July 7, 2026 14:40
@zero4862
zero4862 force-pushed the docs-revision-websockets branch from 85c8f21 to d40178c Compare July 7, 2026 14:55
Comment thread mkdocs/pages/en/devbook/reference/websockets/index.md
Comment thread mkdocs/snippets/devbook/websockets/listen_transaction_flow.py
@zero4862
zero4862 requested a review from segfaultxavi July 7, 2026 15:01
@zero4862
zero4862 marked this pull request as ready for review July 7, 2026 15:06
Comment thread mkdocs/pages/en/devbook/reference/websockets/index.md
@zero4862
zero4862 requested review from Jaguar0625 and removed request for Jaguar0625 July 7, 2026 15:14

@Jaguar0625 Jaguar0625 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.

馃憤

Comment thread mkdocs/pages/en/devbook/reference/websockets/index.md Outdated
Comment thread mkdocs/pages/en/devbook/reference/websockets/index.md Outdated
Comment thread mkdocs/pages/en/devbook/websockets/listen-transaction-flow.md Outdated

@segfaultxavi segfaultxavi 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.

Markdown requires code fences to have blank lines around them. MkDocs is not picky about it, but it triggers my IDE's Markdown linter :)
Also, the style is inconsistent in the file, so please add the blank lines.

Comment thread mkdocs/pages/en/devbook/reference/websockets/index.md Outdated
Comment thread mkdocs/pages/en/devbook/reference/websockets/index.md
Comment thread mkdocs/pages/en/devbook/reference/websockets/index.md Outdated
Comment thread mkdocs/pages/en/devbook/reference/websockets/index.md Outdated
Comment thread mkdocs/pages/en/devbook/reference/websockets/index.md Outdated
Comment thread mkdocs/pages/en/devbook/websockets/listen-transaction-flow.md Outdated
Comment thread mkdocs/pages/en/devbook/websockets/listen-transaction-flow.md Outdated
Comment thread mkdocs/pages/en/devbook/websockets/listen-transaction-flow.md Outdated
Comment thread mkdocs/pages/en/devbook/websockets/listen-transaction-flow.md
@NemProject NemProject deleted a comment from dgarcia360 Jul 8, 2026
@zero4862
zero4862 force-pushed the docs-revision-websockets branch from 13165b2 to 4b45051 Compare July 8, 2026 17:51
@zero4862
zero4862 force-pushed the docs-revision-websockets branch from a14e7a8 to 5180827 Compare July 8, 2026 17:55
@zero4862
zero4862 requested a review from segfaultxavi July 8, 2026 17:55

@Jaguar0625 Jaguar0625 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.

馃憤

@segfaultxavi segfaultxavi 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.

馃憤

@segfaultxavi
segfaultxavi merged commit 02c155f into NemProject:new-docs Jul 9, 2026
10 of 14 checks passed
segfaultxavi pushed a commit that referenced this pull request Jul 9, 2026
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.

3 participants