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

Add Ukrainian localization #2336

Merged
merged 1 commit into from
Jan 3, 2024
Merged

Add Ukrainian localization #2336

merged 1 commit into from
Jan 3, 2024

Conversation

imax9000
Copy link
Contributor

No description provided.

@ansh
Copy link
Contributor

ansh commented Dec 29, 2023

Great, thanks for the contribution!

I will wait for a bit and see if there are any other Ukrainian users who would like to cross-check this translation. Thanks!

Also, would be great if you pull from main and update the merge conflicts till then

@imax9000
Copy link
Contributor Author

Great, thanks for the contribution!

I will wait for a bit and see if there are any other Ukrainian users who would like to cross-check this translation. Thanks!

We've already had a few people going over it on Crowdin.

Also, would be great if you pull from main and update the merge conflicts till then

Will do, give a moment.

@imax9000
Copy link
Contributor Author

Also, would be great if you pull from main and update the merge conflicts till then

Done

@imax9000
Copy link
Contributor Author

Note on the comments w.r.t. localization itself: I started with auto-translation and reviewed and edited all strings in a single pass, with terminology consistency in mind. For the future edits let's please use Crowdin for a couple reasons:

  1. Carrying over the comments from here to there is incredibly annoying and I'm definitely not going to edit .po directly in a text editor.
  2. Preserving context and consistency: once this PR is merged, no one will ever look at it again. But in Crowdin we have the ability to record the decisions regarding terminology as terms, and have them automatically show up whenever a new string that uses them gets added to the source.

Also, I'd much prefer to have this submitted sooner rather than later and send followups with edits later. Because this PR touches a few files that are bottlenecks for merge conflicts, and updating it manually every time line numbers in main change is quite toilsome.

@imax9000 imax9000 force-pushed the uk-l10n branch 3 times, most recently from a25a623 to 4b34cb7 Compare January 2, 2024 14:41
@imax9000
Copy link
Contributor Author

imax9000 commented Jan 2, 2024

@ansh updated once more, still up to date with main. Thanks you @ryvwel for adding your suggestions on Crowdin, now most of them are integrated here.

@ansh
Copy link
Contributor

ansh commented Jan 3, 2024

@imax9000 Can all the conversations be resolved?

@imax9000
Copy link
Contributor Author

imax9000 commented Jan 3, 2024

Yes

@imax9000
Copy link
Contributor Author

imax9000 commented Jan 3, 2024

Want me to do another rebase?

@ansh
Copy link
Contributor

ansh commented Jan 3, 2024

@imax9000 That would be great!

@imax9000
Copy link
Contributor Author

imax9000 commented Jan 3, 2024

@ansh done

@ansh ansh merged commit 401b340 into bluesky-social:main Jan 3, 2024
4 checks passed
@imax9000 imax9000 deleted the uk-l10n branch January 4, 2024 14:10
estrattonbailey added a commit that referenced this pull request Jan 8, 2024
* origin: (45 commits)
  Fix splash config (#2452)
  convert prefix to lowercase in actor autocomplete query (#2431)
  support intl tenor links (#2438)
  Splash: reduce motion + dark mode (#2448)
  Bump android app version code
  Don't use mask for android at all (#2445)
  Update Korean localization (#2432)
  Use android mode, fix fallback (#2437)
  Update for Version 1.63 Português (BR) (#2435)
  New translations messages.po (Ukrainian) (#2422)
  Update Japanese localization (addition + language code correction) (#2423)
  1.63
  Additional embed sources and external-media consent controls (#2424)
  Reduce web requests (#2420)
  New user home feed fixes (#2421)
  E2E runner fixes (#2428)
  Recompile all locales (#2411) (#2416)
  Add Ukrainian localization (#2336)
  Update Portuguese for Português (#2414)
  Add Portuguese Localization (#2407)
  ...
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.

4 participants