Skip to content

🔄 Sync supportpal/addon-language-english#12

Open
bytestream wants to merge 1 commit intomasterfrom
sync
Open

🔄 Sync supportpal/addon-language-english#12
bytestream wants to merge 1 commit intomasterfrom
sync

Conversation

@bytestream
Copy link
Copy Markdown
Member

@bytestream bytestream commented Jun 9, 2025

Synced files with /addon-language-english.


This PR was created automatically by #15532948543

Copy link
Copy Markdown

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

This PR synchronizes language files and GitHub Actions workflows from the English language addon repository. The changes primarily reorganize translation strings and update CI/CD dependencies.

  • Reorganizes the published_desc translation key in the Spanish selfservice language file to a new version section (5.7.5)
  • Adds a new translation key not_published (currently untranslated)
  • Fixes string interpolation syntax in core.php by removing an unnecessary backslash
  • Updates GitHub Actions checkout action from v4 to v5 across workflow files

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
Lang/es/selfservice.php Moves published_desc key to version 5.7.5 section and adds new not_published key (needs Spanish translation)
Lang/es/core.php Fixes string interpolation by removing unnecessary backslash escape before ${count} variable
.github/workflows/test.yml Updates actions/checkout from v4 to v5 in two locations
.github/workflows/release.yml Updates actions/checkout from v4 to v5

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Lang/es/selfservice.php
"comment_notification" => "Nuevo comentario publicado",
"comment_notification_desc" => "Te notifica cuando se publica un nuevo comentario en un artículo de autoservicio.",

/*
Copy link

Copilot AI Dec 3, 2025

Choose a reason for hiding this comment

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

The newly added string "not_published" on line 170 is in English but should be translated to Spanish. Consider translating "Not Published" to "No Publicado" to match the existing translation pattern where "published" is "Publicado" (line 21).

Copilot uses AI. Check for mistakes.
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