Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix wrong link in new_python_api_cn.md #4742

Merged
merged 1 commit into from
May 16, 2022

Conversation

tiancaishaonvjituizi
Copy link
Contributor

y1s1 这样的错误对于 paddle 来说是不应该出现的。建议做一个自动化的机制保证每一个链接都是正确的

@paddle-bot-old
Copy link

Thanks for your contribution!

@tiancaishaonvjituizi
Copy link
Contributor Author

tiancaishaonvjituizi commented May 12, 2022

文档预览构建完成,请点击查看: http://preview-pr-4742.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/zh/api/index_cn.html

另外这个文档预览链接怎么用呀,我点进去显示页面不存在,难道是一个 “已经坏了很久所以大家都习惯了” 的功能吗

@TCChenlong
Copy link
Collaborator

文档预览构建完成,请点击查看: http://preview-pr-4742.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/zh/api/index_cn.html

另外这个文档预览链接怎么用呀,我点进去显示页面不存在,难道是一个 “已经坏了很久所以大家都习惯了” 的功能吗

使用说明在wiki中,还是 beta 版,需要Docs-New这条流水线顺利跑完:
https://github.com/PaddlePaddle/docs/wiki/%5BBeta%5D%E9%A3%9E%E6%A1%A8%E6%96%87%E6%A1%A3%E9%A2%84%E8%A7%88%E5%B7%A5%E5%85%B7

Copy link
Collaborator

@TCChenlong TCChenlong left a comment

Choose a reason for hiding this comment

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

LGTM, thx~

@tiancaishaonvjituizi
Copy link
Contributor Author

tiancaishaonvjituizi commented May 12, 2022

需要Docs-New这条流水线顺利跑完

我提一些问题和建议供参考哈

  • 这条流水线提示 docs/dev_guides/api_contributing_guides/new_python_api_cn.md 的代码风格错误,是一个老问题吗

  • 那是不是可以在 bot 发的消息中注明 “Docs-New 跑完后即可预览”,现在 bot 的消息是 “文档预览构建完成,请点击查看”,会误导用户(因为实际上构建并没有完成,也不能点击查看)

  • 另外,bot 消息中 “请点击查看” 的后面是半角冒号,改为全角冒号可能会更好一些

@TCChenlong

@TCChenlong
Copy link
Collaborator

需要Docs-New这条流水线顺利跑完

我提一些问题和建议供参考哈

  • 这条流水线提示 docs/dev_guides/api_contributing_guides/new_python_api_cn.md 的代码风格错误,是一个老问题吗
  • 那是不是可以在 bot 发的消息中注明 “Docs-New 跑完后即可预览”,现在 bot 的消息是 “文档预览构建完成,请点击查看”,会误导用户(因为实际上构建并没有完成,也不能点击查看)
  • 另外,bot 消息中 “请点击查看” 的后面是半角冒号,改为全角冒号可能会更好一些

@TCChenlong

1 是这篇文档的问题;我再check下;
2 & 3是很棒的建议,我们尽快调整下~
再次感谢~

@TCChenlong TCChenlong merged commit cf34239 into PaddlePaddle:develop May 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants