Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Ref #3109, Fix #3310: Update Translations #3309

Merged
merged 1 commit into from Feb 12, 2021

Conversation

kylehickinson
Copy link
Collaborator

@kylehickinson kylehickinson commented Feb 12, 2021

Summary of Changes

This pull request references #3109
This pull request fixes #3310

Submitter Checklist:

  • Unit Tests are updated to cover new or changed functionality
  • User-facing strings use NSLocalizableString()

Reviewer Checklist:

  • Issues include necessary QA labels:
    • QA/(Yes|No)
    • release-notes/(include|exclude)
    • bug / enhancement
  • Necessary security reviews have taken place.
  • Adequate unit test coverage exists to prevent regressions.
  • Adequate test plan exists for QA to validate (if applicable).
  • Issue is assigned to a milestone (should happen at merge time).

@kylehickinson kylehickinson added this to the 1.23 milestone Feb 12, 2021
@@ -8,7 +8,7 @@
"BraveRewardsAdNotificationTitle" = "Brave Rewards";

/* No comment provided by engineer. */
"BraveRewardsOpen" = "Otwórz";
"BraveRewardsOpen" = "Otwarte";
Copy link
Collaborator

Choose a reason for hiding this comment

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

this reviewed string is actually incorrect, i should do Polish translations manually at some point, there's few places with incorrect words :D nothing to do here now

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Hah, both translations are bad?

Copy link
Collaborator

Choose a reason for hiding this comment

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

first one is correct, in Poland Open can mean 2 things, 2 slightly different words depending on context

  1. Open as 'a shop is open'
  2. Open as 'open Brave Rewards panel'

@kylehickinson kylehickinson changed the title Ref #3109, Ref #3220: Update Translations Ref #3109, Fix #3310: Update Translations Feb 12, 2021
@kylehickinson kylehickinson merged commit 2e5a466 into development Feb 12, 2021
@kylehickinson kylehickinson deleted the update-translations branch February 12, 2021 22:09
kylehickinson added a commit that referenced this pull request Feb 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

update translations for tooltip
4 participants