Opened on 01/24/2017 at 10:10:54 AM
Closed on 01/25/2017 at 10:11:07 AM
#4836 closed defect (fixed)
Buildtools uses an old URL for checking allowed Crowdin locales
| Reported by: | kzar | Assignee: | kzar |
|---|---|---|---|
| Priority: | P2 | Milestone: | |
| Module: | Automation | Keywords: | |
| Cc: | sebastian, trev, kvas | Blocked By: | |
| Blocking: | Platform: | Unknown / Cross platform | |
| Ready: | yes | Confidential: | no |
| Tester: | Unknown | Verified working: | no |
| Review URL(s): | |||
Description (last modified by trev)
Environment
Buildtools 8a2661537ab7
How to reproduce
- Type ./build.py -t generic setuptrans APIKEY from a repository, for example adblockplusui. (After being careful to alter the basename setting in metadata.genreic so that the real Crowdin project is not used!)
Observed behaviour
An exception is thrown since the http://crowdin.net/page/language-codes page no longer exists.
Expected behaviour
No exception is thrown, the target languages are updated properly.
Notes
- The correct URL seems to be https://crowdin.com/languages/languages_list?callback=.
- Since the old page was HTML and the new one uses an API we will need to modify the regular expression that matches locale names too.
Attachments (0)
Change History (4)
Note: See
TracTickets for help on using
tickets.
A commit referencing this issue has landed:
Issue 4836 - Have the setuptrans command use the current Crowdin API