Skip to content

Translations update from Hosted Weblate#34

Merged
inventory69 merged 2 commits intomainfrom
weblate-translations
Mar 3, 2026
Merged

Translations update from Hosted Weblate#34
inventory69 merged 2 commits intomainfrom
weblate-translations

Conversation

@inventory69
Copy link
Owner

Translations update from Hosted Weblate for Simple Notes Sync/Android-App.

Current translation status:

Weblate translation status

inventory69 added a commit that referenced this pull request Mar 3, 2026
Fix 3 errors in PR Build Check workflow (Run #5, PR #34):

1. Fix SyncProgressTest: SyncPhase now has 9 values (DELETING added
   in v1.10.0-P2 but test not updated, expected 8)
2. Add missing permissions (pull-requests: write) to pr-build-check
   workflow — fixes 403 'Resource not accessible' on PR comments
3. Exclude Weblate translation PRs from pr-build-check (own workflow)

Add Weblate translation review workflow:
- weblate-merge.yml: auto-label, convert to Draft, post review
  checklist with translation coverage stats
- weblate-build-check.yml: build only triggers after maintainer
  approval or 'Ready for Review' — saves CI minutes on incomplete
  translations

Refs: PR #34 (Ukrainian translation by @FromKaniv via Weblate)
@inventory69
Copy link
Owner Author

Hey @FromKaniv, thanks for working on the Ukrainian translation!

I can see you've already translated 50%+ in Weblate, which is great and enough for me to merge. The PR itself still shows an empty strings.xml though, so it seems Weblate hasn't pushed the strings yet.

Could you go to Weblate and hit "Commit" so the translations get pushed to this branch? After that I'll review and merge.

Also just a heads-up: the CI failure here is not related to your translation, it's a pre-existing issue I'm fixing in a separate PR.

inventory69 added a commit that referenced this pull request Mar 3, 2026
- Fix SyncProgressTest: update expected SyncPhase count from 8→9 (DELETING added in v1.10.0)
- Fix permissions: add pull-requests: write to pr-build-check workflow (fixes 403 on PR comments)
- Fix unit tests: change continue-on-error from true to false
- Add Weblate review gate: skip pr-build-check for weblate-translations branch
- New weblate-build-check.yml: build only runs after maintainer approval or ready-for-review
- Extend weblate-merge.yml: auto-label, convert to Draft, post review checklist with translation stats
- Restrict build-debug-apk.yml to manual trigger only
- Translate all workflow texts from German to English

Refs: #34

[skip ci]
@inventory69 inventory69 force-pushed the weblate-translations branch 2 times, most recently from ef24f7b to 08648d9 Compare March 3, 2026 09:41
@github-actions
Copy link

github-actions bot commented Mar 3, 2026

🌍 Translation PR — Review Required

This PR contains translation updates from Weblate.

📊 Translation Coverage (in this PR)

Language Translated Coverage
UK 250 / 495 51%

✅ Review Checklist

Please check the following before approving:

  • Coverage: Is the translation sufficiently complete? (Recommendation: ≥50%)
  • Spot check: Verify 5–10 random strings for correctness (back-translate via Google Translate)
  • Placeholders: Are %s, %d, %1$s etc. correctly preserved?
  • App name: Is app_name unchanged ("Simple Notes")?
  • XML validity: No broken XML tags or encoding errors?
  • Length: Are translations not excessively long (UI overflow)?
  • Consistency: Is terminology used consistently?
  • locales_config.xml: Needs to be updated (add new language)

🔍 How to review translations you don't speak

  1. Back-translate: Copy translated strings → Google Translate / DeepL → compare with English original
  2. Weblate checks: Open Weblate project — built-in quality checks for placeholders, length, duplicates etc.
  3. Community review: Ask in the PR if another speaker of the language can review

⏭️ Next Steps

  1. Review translations (see checklist above)
  2. If OK: Approve the PR or mark it as Ready for Review
  3. CI build starts automatically after approval
  4. After successful build: Merge

⚠️ Note: This PR was automatically marked as Draft so the CI build doesn't run unnecessarily. Mark it as "Ready for Review" when you're done reviewing.

@inventory69 inventory69 force-pushed the weblate-translations branch 2 times, most recently from b8d5afa to b63e2d3 Compare March 3, 2026 10:19
Currently translated at 50.0% (253 of 505 strings)

Translation: Simple Notes Sync/Android-App
Translate-URL: https://hosted.weblate.org/projects/simple-notes-sync/android-app/uk/
@inventory69 inventory69 force-pushed the weblate-translations branch from b63e2d3 to 5a53d66 Compare March 3, 2026 11:03
@github-actions
Copy link

github-actions bot commented Mar 3, 2026

✅ Translation Build successful!

The translation PR was built and tested successfully.

Next step: Perform squash-merge.

Don't forget to update locales_config.xml if a new language was added.

@inventory69 inventory69 merged commit 2637c8c into main Mar 3, 2026
2 checks passed
@inventory69
Copy link
Owner Author

@FromKaniv Thanks for your contribution, it will be included in the next version. :)

@FromKaniv
Copy link
Contributor

@inventory69 Hi! You're very welcome. If there are any updates — I’m ready to keep the translation up to date. Good luck with the app's development!

inventory69 added a commit that referenced this pull request Mar 4, 2026
Thanks to @FromKaniv for the Ukrainian translation (#34, #37)
Thanks to @ksuheyl for the Turkish translation (#36)

<3
inventory69 added a commit that referenced this pull request Mar 4, 2026
Thanks to @FromKaniv for the Ukrainian translation (#34, #37)
Thanks to @ksuheyl for the Turkish translation (#36)

<3
inventory69 added a commit that referenced this pull request Mar 4, 2026
Thanks to @FromKaniv for the Ukrainian translation (#34, #37)
Thanks to @ksuheyl for the Turkish translation (#36)

<3
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.

2 participants