Skip to content

Comments

docs: add the chinese version of application architecture#3189

Merged
lumianph merged 6 commits into4paradigm:mainfrom
TanZiYen:docs_tutorial_zh
Apr 18, 2023
Merged

docs: add the chinese version of application architecture#3189
lumianph merged 6 commits into4paradigm:mainfrom
TanZiYen:docs_tutorial_zh

Conversation

@TanZiYen
Copy link
Contributor

@TanZiYen TanZiYen commented Apr 4, 2023

No description provided.

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Apr 4, 2023

## 全量数据存储于实时数据库(不推荐)

![实时决策1](C:\Users\User\Downloads\OpenMLDB\docs\zh\tutorial\images\实时决策1.png)
Copy link
Collaborator

Choose a reason for hiding this comment

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

  1. use the english filename
  2. change it to relative path, e.g. "images/figure1.png"

Copy link
Collaborator

Choose a reason for hiding this comment

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

apply those two changes to all figures accordingly

Copy link
Collaborator

Choose a reason for hiding this comment

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

also, there should be no space in the file name "common architecture.md" => "common_architecture.md"

@lumianph lumianph changed the title docs_tutorial_zh_common architecture docs: add the chinese version of application architecture Apr 10, 2023

## 全量数据存储于实时数据库(不推荐)

![ca1](C:\Users\65972\Documents\GitHub\OpenMLDB\docs\zh\tutorial\images\ca1.png)
Copy link
Collaborator

Choose a reason for hiding this comment

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

change all paths to like this: images/ca1.png,

@lumianph lumianph merged commit 789cacc into 4paradigm:main Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants