Skip to content

[AC-762] Configure Crowdin to localize watch app#2552

Merged
fedemkr merged 4 commits intomasterfrom
AC-762-localize-watch
Jul 17, 2023
Merged

[AC-762] Configure Crowdin to localize watch app#2552
fedemkr merged 4 commits intomasterfrom
AC-762-localize-watch

Conversation

@fedemkr
Copy link
Member

@fedemkr fedemkr commented May 30, 2023

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

Localize the watch app, therefore setup Crowdin GitHub actions configuration to do so.

Code changes

  • Any Localizable.strings and project.pbxproj: Added all the languages that are supported on the main app.
  • crowdin.yml: Set up watch app localizable files with Crowdin

Before you submit

  • Please check for formatting errors (dotnet format --verify-no-changes) (required)
  • Please add unit tests where it makes sense to do so (encouraged but not required)
  • If this change requires a documentation update - notify the documentation team
  • If this change has particular deployment requirements - notify the DevOps team

@joseph-flinn
Copy link

The crowdin.yml changes look good to me

@bitwarden-bot
Copy link

bitwarden-bot commented Jul 10, 2023

Logo
Checkmarx One – Scan Summary & Detailsaa378b08-190a-4728-a6c2-748149c42a81

No New Or Fixed Issues Found

@fedemkr fedemkr marked this pull request as ready for review July 10, 2023 22:50
@fedemkr fedemkr requested review from a team July 10, 2023 22:50
@fedemkr fedemkr merged commit 6d7970f into master Jul 17, 2023
@fedemkr fedemkr deleted the AC-762-localize-watch branch July 17, 2023 16:35
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.

4 participants