Skip to content

i18n: Add missing Arabic translation for 'Preview Only' in ar/message…#12456

Merged
RayBB merged 4 commits intointernetarchive:masterfrom
hichemferjani:fix/arabic-preview-only-translation
Apr 29, 2026
Merged

i18n: Add missing Arabic translation for 'Preview Only' in ar/message…#12456
RayBB merged 4 commits intointernetarchive:masterfrom
hichemferjani:fix/arabic-preview-only-translation

Conversation

@hichemferjani
Copy link
Copy Markdown
Contributor

@hichemferjani hichemferjani commented Apr 28, 2026

Fixes #12456

What this PR does

Adds the missing Arabic translation for the "Preview Only" button string
in openlibrary/i18n/ar/messages.po.

Changes

  • Added msgid "Preview Only" / msgstr "معاينة فقط" to the Arabic locale file

Testing

  • Verified the entry follows the same format as other translated strings in the file
  • Translation is consistent with the existing Arabic translation for "Preview" (معاينة)

@mekarpeles
Copy link
Copy Markdown
Member

Thank you for this PR, @hichemferjani, and welcome to Open Library — this looks like your first contribution!

🤖 Copilot has been assigned for an initial review.

A reviewer must first be assigned. There are currently 65 open PRs of equal or higher priority ahead of yours.

PR triage checklist (maintainers / Pam)
  • PR description — not empty; explains what the change does and how to verify it
  • References an issue — PR body contains a #NNN reference
    • Linked issue is triaged — has a Priority: * label (not just Needs: Triage)
    • Linked issue is assigned — has at least one assignee
  • Commit history clean — no WIP/fixup/conflict noise; commit messages are meaningful
  • CI passing — no failing check-runs
  • Test cases present — if the change touches substantive logic, test coverage exists or is explained
  • Proof of testing — PR body includes a description of what was tested, a screenshot, or a video

Note

This comment was automatically generated by Pam, Open Library's Project AI Manager, on behalf of @mekarpeles. Pam is designed to provide status visibility, perform basic project management functions and relevant codebase research, and provide actionable feedback so contributors aren't left waiting.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds the missing Arabic translation for the “Preview Only” UI string so Arabic users see a localized label in the book preview CTA.

Changes:

  • Added an Arabic msgstr for msgid "Preview Only" in the Arabic PO catalog.

Comment thread openlibrary/i18n/ar/messages.po Outdated
hichemferjani and others added 2 commits April 28, 2026 12:27
Comment thread openlibrary/i18n/ar/messages.po
Copy link
Copy Markdown
Collaborator

@RayBB RayBB left a comment

Choose a reason for hiding this comment

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

Tossed on google translate and it seems correct.
Looks good to me.

@RayBB RayBB merged commit c1d79b5 into internetarchive:master Apr 29, 2026
3 checks passed
@tfmorris
Copy link
Copy Markdown
Contributor

I totally don't understand how the process works here. This PR doesn't reference a valid issue. The issue that it SHOULD reference (#12455) hasn't been triaged yet, has no lead assigned, and there's an ongoing discussion there about what the best approach is.

Why is there sometimes draconian enforcement of process and at other times it's completely ignored?

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.

5 participants