Skip to content

Conversation

@KaylaBrady
Copy link
Collaborator

Summary

What is this PR for?

iOS

  • If you added any user-facing strings on iOS, are they included in Localizable.xcstrings?
    • Add temporary machine translations, marked "Needs Review"

android

  • All user-facing strings added to strings resource in alphabetical order
  • Expensive calculations are run in withContext(Dispatchers.Default) where possible (ideally in shared code)

Testing

What testing have you done?

@KaylaBrady KaylaBrady requested a review from a team as a code owner July 22, 2025 15:38
@KaylaBrady KaylaBrady requested a review from EmmaSimon July 22, 2025 15:38
@KaylaBrady KaylaBrady added this pull request to the merge queue Jul 22, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 22, 2025
@KaylaBrady KaylaBrady added this pull request to the merge queue Jul 22, 2025
Merged via the queue into main with commit 9ab797f Jul 22, 2025
7 checks passed
@KaylaBrady KaylaBrady deleted the kb-ios-1.2.7 branch July 22, 2025 17:17
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.

3 participants