Skip to content

Conversation

@echonesis
Copy link
Contributor

Purpose of the pull request

Related: #443

What's changed?

  • translate all files in Chapter 3 (Write) from CN to EN
  • test locally by command: yarn start

Checklist

  • I have read the CONTRIBUTING guide.
  • I have written the necessary doc or comment.

@delei delei self-requested a review August 13, 2025 02:38
Comment comment = drawingPatriarch.createCellComment(
new XSSFClientAnchor(0, 0, 0, 0, (short) 1, 0, (short) 2, 1));
comment.setString(new XSSFRichTextString("批注内容"));
comment.setString(new XSSFRichTextString("Comment content"));
Copy link
Member

Choose a reason for hiding this comment

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

As for now, in the sample code, only the comments need to be translated, the code values do not need to be modified, so that they remain consistent with the images.

Similar content in other documents does not need to be translated either.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks @delei
They're modifed in the updated version.

@echonesis echonesis requested a review from delei August 13, 2025 05:58
Copy link
Member

@delei delei left a comment

Choose a reason for hiding this comment

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

LGTM

@delei delei merged commit eaa944e into apache:main Aug 13, 2025
5 checks passed
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.

2 participants