🔄 Sync supportpal/addon-language-english#7
Open
bytestream wants to merge 1 commit into
Open
Conversation
bytestream
force-pushed
the
sync
branch
6 times, most recently
from
534ae3c to
c776191
Compare
There was a problem hiding this comment.
Pull request overview
This PR syncs language files and workflow configurations from the English language addon repository. The changes include updated translations for version 5.7.0, relocated existing translation keys, and GitHub Actions dependency updates.
- Updated Portuguese translation files with new strings for version 5.7.0 features including operator notifications, reply templates, and ticket actions
- Reorganized existing translation keys by moving them to appropriate version sections
- Updated GitHub Actions dependencies from v4 to v5 for checkout actions and from 9.1 to 9.5 for phplint
Reviewed changes
Copilot reviewed 15 out of 15 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| Lang/pt/user.php | Added 31 new translation strings for operator notifications and subscription settings (5.7.0); removed outdated notification-related strings |
| Lang/pt/ticket.php | Added 49 new translation strings for notifications, rate limiting, and ticket actions (5.5.0-5.7.0); relocated allowed_files_desc to proper version section |
| Lang/pt/selfservice.php | Added new comment notification strings (5.7.0); relocated published_desc and added not_published (5.7.5) |
| Lang/pt/report.php | Added group_by translation string (5.7.0) |
| Lang/pt/permission.php | Added permission strings for operator notifications and reply templates (5.7.0) |
| Lang/pt/messages.php | Added throttle_error message string (5.4.1) |
| Lang/pt/log.php | Added log strings for pinned messages and operator notification updates (5.5.0, 5.7.0) |
| Lang/pt/locale.php | Repositioned Europe/Istanbul timezone entry (ordering adjustment) |
| Lang/pt/license.php | Added key_updated success message (5.5.0) |
| Lang/pt/integrations.php | Relocated Twilio verification strings and added new integration settings for Algolia and Microsoft (5.4.1-5.7.0) |
| Lang/pt/general.php | Added general UI strings including mark_as_done, bot, and allow_all (5.5.0, 5.7.0) |
| Lang/pt/core.php | Added extensive UI strings for table editing, notifications, and cursor positioning (5.5.0-5.7.3); relocated existing strings |
| Lang/pt/actions.php | Removed close_and_lock_ticket; added separate close_ticket, lock_ticket, and notify_user actions (5.5.0) |
| .github/workflows/test.yml | Updated GitHub Actions dependencies: checkout@v4→v5, phplint@9.1→9.5 |
| .github/workflows/release.yml | Updated checkout action from v4 to v5 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "exclude_addresses" => "Exclude Addresses", | ||
| "exclude_addresses_desc" => "A list of email addresses which are excluded from rate limits.", | ||
| "enable_throttling" => "Enable Throttling", | ||
| "enable_throttling_api_desc" => "Limit the number of messages user's can post within a time frame. See <a href='https://docs.supportpal.com/current/App+Configuration#Throttling' target='_blank'>Throttling configuration</a> for more details on how to adjust the limits.", |
There was a problem hiding this comment.
Grammar error: "user's" should be "users" (plural, not possessive). The sentence should read "Limit the number of messages users can post within a time frame."
Suggested change
| "enable_throttling_api_desc" => "Limit the number of messages user's can post within a time frame. See <a href='https://docs.supportpal.com/current/App+Configuration#Throttling' target='_blank'>Throttling configuration</a> for more details on how to adjust the limits.", | |
| "enable_throttling_api_desc" => "Limit the number of messages users can post within a time frame. See <a href='https://docs.supportpal.com/current/App+Configuration#Throttling' target='_blank'>Throttling configuration</a> for more details on how to adjust the limits.", |
| "Europe/Vilnius" => "(GMT+02:00) Vilnius", | ||
| "Africa/Windhoek" => "(GMT+02:00) Windhoek", | ||
| "Asia/Baghdad" => "(GMT+03:00) Baghdad", | ||
| "Europe/Istanbul" => "(GMT+02:00) Istanbul", |
There was a problem hiding this comment.
The timezone entry for "Europe/Istanbul" appears to be misplaced. It's positioned after "Asia/Baghdad" (GMT+03:00) but is marked as (GMT+02:00), which breaks the ordering by timezone offset. This entry should remain with other GMT+02:00 timezones (around line 312-320) to maintain consistent ordering.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Synced files with /addon-language-english.
This PR was created automatically by #11385185308