Skip to content
This repository was archived by the owner on Jun 24, 2025. It is now read-only.

i18n support again (part 1) - #264

Merged
eliandoran merged 69 commits into
developfrom
feature/i18n
Aug 12, 2024
Merged

i18n support again (part 1)#264
eliandoran merged 69 commits into
developfrom
feature/i18n

Conversation

@Nriver

@Nriver Nriver commented Jul 24, 2024

Copy link
Copy Markdown
Collaborator

I'll continue work on i18n here. As #257 was closed accidently.

@Nriver
Nriver marked this pull request as ready for review July 24, 2024 01:52
@Nriver
Nriver marked this pull request as draft July 24, 2024 01:59
@eliandoran
eliandoran force-pushed the develop branch 3 times, most recently from eff61af to 4090386 Compare August 10, 2024 11:14
@eliandoran

eliandoran commented Aug 11, 2024

Copy link
Copy Markdown
Contributor

@Nriver , I think we should integrate the first batch of translations before we proceed any further.

Great work so far, by the way.

@eliandoran

eliandoran commented Aug 11, 2024

Copy link
Copy Markdown
Contributor

@Nriver , I did the merge on my side since I was familiar with the changes that I did.
Please update the Chinese translation to match the changes in 3a837cf#diff-5e63e49255e864d2949f4c4e70bda258343bbec26c5f09e9d371757b6ce2c765

@eliandoran eliandoran left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks really good so far, we can merge it as soon as the Chinese translations are updated.

@eliandoran eliandoran changed the title i18n support again i18n support again (part 1) Aug 11, 2024
@eliandoran eliandoran added this to the v0.90.5-beta milestone Aug 11, 2024
@Nriver

Nriver commented Aug 12, 2024

Copy link
Copy Markdown
Collaborator Author

Ok, just updated the links in the Chinese translation file.

@Nriver
Nriver marked this pull request as ready for review August 12, 2024 01:55

@eliandoran eliandoran left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants