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

Ukrainian translations were added and improved #20709

Merged
merged 11 commits into from
Sep 30, 2024

Conversation

vlavrynovych
Copy link
Contributor

I reviewed all the previously existed lines and improved many of them. For the rest I just filled what was missed. Still probably not ideal because it's not alway clear what is the context or where it will be displayed. Like "Edit". If it's an button it should be "Радагувати", if it's a command for action then it should sound like "Редагуй". So, I selected the most common options where it was possible.

⚠️ I didn't check if the JSON structure is the latest version or not. Just filled what I had in the main branch at the moment

@cathysarisky
Copy link
Contributor

Hey @vlavrynovych , thank you for these!

There are a few new strings for portal available and also the ones related to search that are new since you submitted these. Would you have time to add them?

You can bring your branch up to date by following these directions: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

* main: (411 commits)
  Shipped comments-ui@0.18.2
  Fixed various visual details in Comments UI (TryGhost#21121)
  Shipped comments-ui@0.18.1
  Update dependency mysql2 to v3.11.3 [SECURITY]
  Added reply button to comment replies
  Update dependency mysql2 to v3.11.2 [SECURITY]
  Fixed flaky member expertise test
  Update dependency mysql2 to v3.11.1 [SECURITY]
  Increased parallelisation of Nx processes
  Update tiptap monorepo to v2.7.2
  Fixed Article feature image not showing up in the feed
  Fixed requiring passing tests for canary builds
  Updated Nx to v19
  🐛 Fixed infinite loops in `setFeatureImageCaption` for deleted posts (TryGhost#21081)
  Fixed versioning and npm visibility for internal apps
  Update dependency html-validate to v8.24.0
  Implemented lazy loading for Inbox & Activities (TryGhost#21106)
  Added sentry logging to local revisions service for localStorage errors (TryGhost#21078)
  Added posts restore UI (TryGhost#21096)
  Fixed bounce rate on stats page (TryGhost#21097)
  ...

# Conflicts:
#	ghost/i18n/locales/uk/comments.json
#	ghost/i18n/locales/uk/portal.json
@vlavrynovych
Copy link
Contributor Author

Hello @cathysarisky, working on it right now...

@vlavrynovych
Copy link
Contributor Author

Hi again @cathysarisky, done. Let me know if any other help is needed.

ghost/i18n/locales/uk/comments.json Outdated Show resolved Hide resolved
ghost/i18n/locales/uk/comments.json Outdated Show resolved Hide resolved
ghost/i18n/locales/uk/comments.json Outdated Show resolved Hide resolved
@vlavrynovych
Copy link
Contributor Author

@cathysarisky, sorry for the long response time. It was a night for me. But anyway I left a comments and commited the what I believe will work in these cases.

@vlavrynovych
Copy link
Contributor Author

I also want to reference other two pull requests which can be used as an additional source:

Copy link
Contributor

@cathysarisky cathysarisky left a comment

Choose a reason for hiding this comment

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

Thanks so much for your work on this! It looks ready to merge to me! :)

@9larsons 9larsons merged commit 8079448 into TryGhost:main Sep 30, 2024
21 checks passed
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.

3 participants