Opened 4 years ago

Closed 4 years ago

#2977 closed change (fixed)

Don't add translations that are identical to the default locale's strings

Reported by: fhd Assignee: fhd
Priority: P2 Milestone: Adblock-Browser-for-Android-beta-1.0.0.2015090419
Module: Adblock-Browser-for-Android Keywords:
Cc: Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

https://codereview.adblockplus.org/29325540/

Description (last modified by fhd)

Background

translations.py get currently downloads all translations from Crowdin, including those where not a single string has been translated. Those end up being identical to the default locale, and since we fall back to en-US anyway, they're just redundant.

What to change

Make translations.py get not add translations that are identical to the default locale's strings.

Change History (5)

comment:1 Changed 4 years ago by fhd

  • Owner set to fhd
  • Priority changed from Unknown to P2
  • Ready set

comment:2 Changed 4 years ago by fhd

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:3 Changed 4 years ago by fhd

  • Description modified (diff)
  • Summary changed from Don't add translations that are identical to the source texts to Don't add translations that are identical to the default locale texts

comment:4 Changed 4 years ago by fhd

  • Summary changed from Don't add translations that are identical to the default locale texts to Don't add translations that are identical to the default locale's strings

comment:5 Changed 4 years ago by fhd

  • Milestone set to Adblock-Browser-for-Android-beta-next
  • Resolution set to fixed
  • Status changed from reviewing to closed
Note: See TracTickets for help on using tickets.