Skip to content

Comments

🔄 Sync supportpal/addon-language-english#6

Open
bytestream wants to merge 1 commit intomasterfrom
sync
Open

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

Conversation

@bytestream
Copy link
Member

Synced files with /addon-language-english.


This PR was created automatically by #11385185308

Copy link

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 from the English addon repository to the Portuguese (Brazil) translation repository. It adds new translation strings, reorganizes existing entries, and updates GitHub Actions workflow dependencies.

Key Changes:

  • Updates GitHub Actions dependencies (checkout@v4 → v5, phplint@9.1 → 9.5)
  • Adds new translation strings for version 5.7.0 features including notification settings, operator notifications, department subscriptions, and various UI elements
  • Reorganizes timezone entries in locale configuration
  • Refactors ticket actions by splitting "close_and_lock_ticket" into separate "close_ticket" and "lock_ticket" actions
  • Moves several translation strings to more appropriate version sections

Reviewed changes

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

Show a summary per file
File Description
.github/workflows/test.yml Updates GitHub Actions checkout to v5 and phplint to 9.5
.github/workflows/release.yml Updates GitHub Actions checkout to v5
Lang/pt_BR/actions.php Removes combined close/lock action, adds separate close, lock, and notify actions
Lang/pt_BR/core.php Adds table editor UI strings, removes obsolete ticket_number_missing entry, reorganizes version sections
Lang/pt_BR/general.php Adds "mark_as_done", "bot", and "allow_all" translations
Lang/pt_BR/integrations.php Reorganizes Twilio and Algolia settings, adds Microsoft tenant ID configuration
Lang/pt_BR/license.php Adds license key update success message
Lang/pt_BR/locale.php Moves Istanbul timezone entry to maintain better ordering
Lang/pt_BR/log.php Adds log entries for pinned messages and operator notification updates
Lang/pt_BR/messages.php Adds throttle error message
Lang/pt_BR/permission.php Adds permissions for operator notifications and reply templates
Lang/pt_BR/report.php Adds "group_by" translation
Lang/pt_BR/selfservice.php Reorganizes published_desc entry, adds comment notification strings
Lang/pt_BR/ticket.php Adds rate limiting, notification, and ticket action translations; reorganizes allowed_files_desc
Lang/pt_BR/user.php Removes old notification settings, adds comprehensive operator notification configuration strings

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

Comment on lines 321 to +322
"Europe/Vilnius" => "(GMT+02:00) Vilnius",
"Africa/Windhoek" => "(GMT+02:00) Windhoek",
"Asia/Baghdad" => "(GMT+03:00) Baghdad",
"Europe/Istanbul" => "(GMT+02:00) Istanbul",
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 timezone "Europe/Istanbul" (GMT+02:00) is placed after "Asia/Baghdad" (GMT+03:00), breaking the logical ordering of timezones by GMT offset. This makes the list harder to maintain and can confuse users. Consider keeping "Europe/Istanbul" with the other GMT+02:00 timezones (before line 321) to maintain consistent ordering by timezone offset.

Suggested change
"Asia/Baghdad" => "(GMT+03:00) Baghdad",
"Europe/Istanbul" => "(GMT+02:00) Istanbul",
"Europe/Istanbul" => "(GMT+02:00) Istanbul",
"Asia/Baghdad" => "(GMT+03:00) Baghdad",

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.

1 participant