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

i18n updates and fixes | fonts to export PDF #446

Merged
merged 13 commits into from
May 4, 2023

Conversation

Cs4K1Sr4C
Copy link
Contributor

[+] SorryDialog translation added on all the existing languages

[fix] Button translation fixes in index.tsx

[+] Chinese, Japanese and Russian fonts added for the PDF export

@vercel
Copy link

vercel bot commented May 3, 2023

@Cs4K1Sr4C is attempting to deploy a commit to the reworkd Team on Vercel.

A member of the Team first needs to authorize it.

@Cs4K1Sr4C
Copy link
Contributor Author

Hi @awtkns, can you please take a closer look on this PR? I have completed the missing translations and I added some fonts for the PDF export method.

@vercel
Copy link

vercel bot commented May 4, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
agent-gpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 4, 2023 7:21pm

@awtkns
Copy link
Member

awtkns commented May 4, 2023

Amazing work @Cs4K1Sr4C. Your are a true i18n star! ⭐

@awtkns awtkns merged commit bfde9fd into reworkd:main May 4, 2023
@Cs4K1Sr4C
Copy link
Contributor Author

Thanks @awtkns ! 🙏💪

There is an issue when the language changed, in the SettingsDialog Mode dropdown doesn't change the options to the selected language, but I'm going to fix that in the next PR, and also I'll add more additional fonts especially to Slovakian, Ukrainian and for the other special languages.

@Cs4K1Sr4C Cs4K1Sr4C deleted the additional-fonts branch May 4, 2023 20:51
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