Skip to content

Update Missing Key from Indonesian Translation#769

Merged
killergerbah merged 2 commits intokillergerbah:mainfrom
yaaacha:main
Sep 16, 2025
Merged

Update Missing Key from Indonesian Translation#769
killergerbah merged 2 commits intokillergerbah:mainfrom
yaaacha:main

Conversation

@yaaacha
Copy link
Copy Markdown
Contributor

@yaaacha yaaacha commented Sep 3, 2025

No description provided.

@yaaacha
Copy link
Copy Markdown
Contributor Author

yaaacha commented Sep 3, 2025

FYI: GitHub Actions build failed on my fork at yarn workspace @project/extension run zip. This looks unrelated to the translation update, might be environment/workflow related(?)

image

Copy link
Copy Markdown
Owner

@killergerbah killergerbah left a comment

Choose a reason for hiding this comment

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

Thanks for the amazing work
Just a couple things to fix and the verify job should pass

"adjustOffset": "Sesuaikan offset takarir ±100 ms",
"adjustOffsetToNextSubtitle": "Sesuaikan offset takarir agar takarir berikutnya berada di waktu saat ini",
"increaseSubtitlePositionOffset": "Tambah Offset Posisi Takarir",
"increaseTopSubtitlePositionOffset": "Tambah Offset Posisi Takarir Atas"
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Missing comma at end of line

"binds": {
"adjustOffset": "Sesuaikan offset takarir ±100 ms",
"adjustOffsetToNextSubtitle": "Sesuaikan offset takarir agar takarir berikutnya berada di waktu saat ini",
"increaseSubtitlePositionOffset": "Tambah Offset Posisi Takarir",
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Some key names changed. You'll need to remove the old ones and add the new ones:

  • increaseSubtitlePositionOffset -> moveBottomSubtitlesUp
  • decreaseSubtitlePositionOffset -> moveBottomSubtitlesDown
  • increaseTopSubtitlePositionOffset -> moveTopSubtitlesUp
  • decreaseTopSubtitlePositionOffset -> moveTopSubtitlesDown

Copy link
Copy Markdown
Contributor Author

@yaaacha yaaacha left a comment

Choose a reason for hiding this comment

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

checked and resolved, thanks for your feedback

@killergerbah killergerbah merged commit ada1581 into killergerbah:main Sep 16, 2025
1 check passed
@killergerbah
Copy link
Copy Markdown
Owner

@yaaacha Thanks again!

@yaaacha
Copy link
Copy Markdown
Contributor Author

yaaacha commented Sep 16, 2025

my pleasure😇

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