tools: update DM TOC and refine content #869
Conversation
README.md
Outdated
| - [Deploy](tools/data-migration-deployment.md) | ||
| - [Synchronize Data](tools/data-migration-practice.md) | ||
| - [Synchronize](tools/data-migration-practice.md) |
There was a problem hiding this comment.
is there a more meaningful name?
There was a problem hiding this comment.
@csuzhangxc Do you have any suggestions on this? :)
There was a problem hiding this comment.
Haha~ Thanks for your suggestion, but I think it might be a little bit weird to use "Get Started" in this position. @GregoryIan What's your opinion and suggestion?
There was a problem hiding this comment.
it's a step by step instructions using dm-ansible. it should belong to Get Started
There was a problem hiding this comment.
I think this file contains deoployment and how to synchroniza data, Get Started may be better
There was a problem hiding this comment.
I want to add a step by step instrucation that contains deploy dm manually and create a simple task, like this file do
There was a problem hiding this comment.
@GregoryIan Perhaps just leave a Deploy in the TOC that points to the tools/data-migration-practice.md file? That file about deploying the DM cluster using DM-Ansible is also mentioned in this file.
There was a problem hiding this comment.
how about
[get started](tools/data-migration-practice.md)
Deploy
- [ansible](tools/data-migration-deployment.md)
- [binary](xxxxx)
What I can confirm is that we have to need a get started. If the content of tools/data-migration-practice.md is not a good enough, we can modify it.
|
LGTM |
@GregoryIan @csuzhangxc @CaitinChen PTAL