Skip to content

Translate to Ukrainian#7961

Merged
Alkarex merged 1 commit intoFreshRSS:edgefrom
nykula:edge
Sep 14, 2025
Merged

Translate to Ukrainian#7961
Alkarex merged 1 commit intoFreshRSS:edgefrom
nykula:edge

Conversation

@nykula
Copy link
Contributor

@nykula nykula commented Sep 14, 2025

Closes #6982

Changes proposed in this pull request:

  • Generate app/i18n/uk using make i18n-add-language and translate every script
  • Add gen.lang.uk using make i18n-add-key and mark it as IGNORE in all languages

How to test the feature manually:

  1. sudo apt install -y php-{cli,curl,mbstring,sqlite3,xml}
  2. git clone https://github.com/nykula/FreshRSS &&cd FreshRSS/p &&php -S 0.0.0.0:8080
  3. Choose Українська in the language section, continue to install and use

Pull request checklist:

  • clear commit messages
  • code manually tested
  • unit tests written (optional if too hard)
  • documentation updated

Additional information can be found in the documentation.

@Inverle
Copy link
Member

Inverle commented Sep 14, 2025

php -S 0.0.0.0:8080

By the way you can use make start to run the apache2 webserver instead, which is better for testing

@nykula
Copy link
Contributor Author

nykula commented Sep 14, 2025

Nice to know! Found and fixed one more typo when trying Apache that way.

One localization-related thing I noticed but didn't touch was the docs/i18n/flags/gen directory. Do I need to initialize something there or does it get populated automatically?

@Inverle
Copy link
Member

Inverle commented Sep 14, 2025

Do I need to initialize something there or does it get populated automatically?

Run make fix-all or ./cli/check.translation.php --generate-readme (short version: make readme)

@nykula
Copy link
Contributor Author

nykula commented Sep 14, 2025

Thanks, ran fix-all and amended with the new svg and readme update.

@Alkarex Alkarex added this to the 1.27.1 milestone Sep 14, 2025
@Alkarex
Copy link
Member

Alkarex commented Sep 14, 2025

FYI, can also be tested in our Dev Container, either online or locally

Open in GitHub Codespaces (link only for the PR owner)

@Alkarex Alkarex merged commit 6d0bec1 into FreshRSS:edge Sep 14, 2025
1 check passed
@Alkarex
Copy link
Member

Alkarex commented Sep 14, 2025

Nice work, thanks!
Please add a line for you in https://github.com/FreshRSS/FreshRSS/blob/edge/CREDITS.md

nykula added a commit to nykula/FreshRSS that referenced this pull request Sep 14, 2025
nykula added a commit to nykula/FreshRSS that referenced this pull request Sep 14, 2025
@nykula nykula mentioned this pull request Sep 14, 2025
4 tasks
aledeg added a commit that referenced this pull request Sep 14, 2025
Follow-up to #7961.

Co-authored-by: Alexis Degrugillier <aledeg@users.noreply.github.com>
@Inverle
Copy link
Member

Inverle commented Sep 14, 2025

link only for the PR owner

why?

@Alkarex
Copy link
Member

Alkarex commented Sep 14, 2025

link only for the PR owner

why?

I got an error Oh no, it looks like you are offline and I thought it was due to owner difference, but actually it also fails for me on the main repo, so there must be another problem - I will investigate another time

@Frenzie
Copy link
Member

Frenzie commented Sep 14, 2025

FYI, can also be tested in our Dev Container, either online or locally

Open in GitHub Codespaces (link only for the PR owner)

Does everyone see their own username there btw?

image

@Inverle
Copy link
Member

Inverle commented Sep 14, 2025

Yes (I have used a codespace for a PR that isn't my own, and it worked, that's why I asked above)

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

Labels

I18n 🌍 Translations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[i18n] Translation Ukrainian (UKR) 🇺🇦

5 participants