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

Translation of l10n/po/zh_CN/_posts/2022-01-13-quarkus-aws-cloudwatch_extension.adoc.po #122

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

xinhuagu
Copy link
Contributor

@xinhuagu xinhuagu commented Aug 12, 2022

Fix #121

@xinhuagu xinhuagu closed this Aug 12, 2022
@xinhuagu xinhuagu changed the title (#121) Translation of l10n/po/zh_CN/_posts/2022-01-13-quarkus-aws-cloudwatch_extension.adoc.po Translation of l10n/po/zh_CN/_posts/2022-01-13-quarkus-aws-cloudwatch_extension.adoc.po Aug 12, 2022
@xinhuagu xinhuagu reopened this Aug 12, 2022
@github-actions
Copy link

Site preview(cn): https://pr-preview-122-cn-quarkusio.surge.sh

1 similar comment
@github-actions
Copy link

Site preview(cn): https://pr-preview-122-cn-quarkusio.surge.sh

@github-actions
Copy link

Site preview(cn): https://pr-preview-122-cn-quarkusio.surge.sh

@zhfeng
Copy link
Collaborator

zhfeng commented Aug 15, 2022

@xinhuagu please check the site preview: https://pr-preview-122-cn-quarkusio.surge.sh/blog/quarkus-aws-cloudwatch_extension/

It looks like we are missing some css style. I will check.

@xinhuagu xinhuagu marked this pull request as draft August 15, 2022 07:15
@github-actions
Copy link

Site preview(cn): https://pr-preview-122-cn-quarkusio.surge.sh

@xinhuagu
Copy link
Contributor Author

@zhfeng I translated the layout msgid, that is why the css is missing. (The layout must be 'post' ) Thanks for pointing this problem out, I've fixed it.

https://pr-preview-122-cn-quarkusio.surge.sh/blog/quarkus-aws-cloudwatch_extension/

@xinhuagu xinhuagu marked this pull request as ready for review August 15, 2022 12:58
@zhfeng
Copy link
Collaborator

zhfeng commented Aug 16, 2022

Thanks for your contribution! and the blog pages looks weird https://pr-preview-122-cn-quarkusio.surge.sh/blog/page/5/ which seems looking for . as the end of the first paragraph. I need more investigating. Can you take a look at translating some guides currently?

@xinhuagu
Copy link
Contributor Author

@zhfeng yes, probably becaus of the chinese full stop... Since the jap version has the same problem : https://ja.quarkus.io/blog/page/5/

ok, I will pick up some guides to translate first. Thanks.

@zhfeng
Copy link
Collaborator

zhfeng commented Aug 19, 2022

Please hold this PR and we wil investigate the blog /page issue.

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.

Translate l10n/po/zh_CN/_posts/2022-01-13-quarkus-aws-cloudwatch_extension.adoc.po
2 participants