Skip to content

[fix][scripts]: prepare_docs.sh supports image path conversion.#789

Merged
zhongjiajie merged 4 commits intoapache:masterfrom
huangchenguang123:fix/fix_scripts
May 31, 2022
Merged

[fix][scripts]: prepare_docs.sh supports image path conversion.#789
zhongjiajie merged 4 commits intoapache:masterfrom
huangchenguang123:fix/fix_scripts

Conversation

@huangchenguang123
Copy link
Copy Markdown
Contributor

@huangchenguang123 huangchenguang123 commented May 20, 2022

Since viewing md on github requires a relative path, use '../../../../'.

so it will convert img path and make them work bot github markdown and website.

see my code in apache/dolphinscheduler#10127

@huangchenguang123
Copy link
Copy Markdown
Contributor Author

Please review my code @zhongjiajie .Thanks.

@zhongjiajie
Copy link
Copy Markdown
Member

Did you test locally? @huangchenguang123

@huangchenguang123
Copy link
Copy Markdown
Contributor Author

Did you test locally? @huangchenguang123

Yes, I tested locally. Is there something wrong with the code?

@zhongjiajie
Copy link
Copy Markdown
Member

No, I just make sure about it, but is seem we do not change the img path in apache/dolphinscheculer, should we also change the path related path in directory docs in apache/dolphinscheculer?

@huangchenguang123
Copy link
Copy Markdown
Contributor Author

No, I just make sure about it, but is seem we do not change the img path in apache/dolphinscheculer, should we also change the path related path in directory docs in apache/dolphinscheculer?

If the documentation wants to be previewed in github/idea, the image path in docs should be changed. like apache/dolphinscheduler#10127. To be compatible with the website, the path needs to be rewritten in the script.From '../../../../img' to '/img'.I don't know if you understand what I mean?

@huangchenguang123
Copy link
Copy Markdown
Contributor Author

Did I should do someting more? @zhongjiajie

@zhongjiajie
Copy link
Copy Markdown
Member

@huangchenguang123 It is my bad, I am working hard on the issue of the 3.0.0-beta-1 release process. I will take a look at this issue after the version release.

@zhongjiajie zhongjiajie self-requested a review May 26, 2022 02:10
@huangchenguang123
Copy link
Copy Markdown
Contributor Author

@huangchenguang123 It is my bad, I am working hard on the issue of the 3.0.0-beta-1 release process. I will take a look at this issue after the version release.

Thanks.

@zhongjiajie
Copy link
Copy Markdown
Member

@huangchenguang123 It is my bad, I am working hard on the issue of the 3.0.0-beta-1 release process. I will take a look at this issue after the version release.

Thanks.

will task a look tonight or tomorrow, we need more local tests about it

@zhongjiajie
Copy link
Copy Markdown
Member

I have checked this PR and it works fine except some comment issues and I have already fixed them in
9703931

Copy link
Copy Markdown
Member

@zhongjiajie zhongjiajie left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@zhongjiajie zhongjiajie merged commit 0beba7d into apache:master May 31, 2022
@zhongjiajie zhongjiajie added the develop Develop section change label May 31, 2022
@zhongjiajie
Copy link
Copy Markdown
Member

Well done and thanks @huangchenguang123

zhongjiajie added a commit to zhongjiajie/dolphinscheduler that referenced this pull request Jun 1, 2022
Use related path in our docs for imgs, previous
can not use because website need the absolute path
from root directory, and after we merged
apache/dolphinscheduler-website#789 we have covert
function to do that

close: apache#9426
zhongjiajie added a commit to zhongjiajie/dolphinscheduler that referenced this pull request Jun 1, 2022
Use related path in our docs for imgs, previous
can not use because website need the absolute path
from root directory, and after we merged
apache/dolphinscheduler-website#789 we have covert
function to do that

close: apache#9426
zhongjiajie added a commit to apache/dolphinscheduler that referenced this pull request Jun 2, 2022
Use related path in our docs for imgs, previous
can not use because website need the absolute path
from root directory, and after we merged
apache/dolphinscheduler-website#789 we have covert
function to do that

close: #9426
ITBOX-ITBOY pushed a commit to ITBOX-ITBOY/dolphinscheduler that referenced this pull request Jul 8, 2022
Use related path in our docs for imgs, previous
can not use because website need the absolute path
from root directory, and after we merged
apache/dolphinscheduler-website#789 we have covert
function to do that

close: apache#9426
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

develop Develop section change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants