{"action":"opened","issue":{"url":"https://api.github.com/repos/LLK/scratch-www/issues/1102","repository_url":"https://api.github.com/repos/LLK/scratch-www","labels_url":"https://api.github.com/repos/LLK/scratch-www/issues/1102/labels{/name}","comments_url":"https://api.github.com/repos/LLK/scratch-www/issues/1102/comments","events_url":"https://api.github.com/repos/LLK/scratch-www/issues/1102/events","html_url":"https://github.com/LLK/scratch-www/issues/1102","id":194787463,"number":1102,"title":"Missing translations: Duplicate strings in the same l10n.json file","user":{"login":"chrisgarrity","id":399209,"avatar_url":"https://avatars.githubusercontent.com/u/399209?v=3","gravatar_id":"","url":"https://api.github.com/users/chrisgarrity","html_url":"https://github.com/chrisgarrity","followers_url":"https://api.github.com/users/chrisgarrity/followers","following_url":"https://api.github.com/users/chrisgarrity/following{/other_user}","gists_url":"https://api.github.com/users/chrisgarrity/gists{/gist_id}","starred_url":"https://api.github.com/users/chrisgarrity/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chrisgarrity/subscriptions","organizations_url":"https://api.github.com/users/chrisgarrity/orgs","repos_url":"https://api.github.com/users/chrisgarrity/repos","events_url":"https://api.github.com/users/chrisgarrity/events{/privacy}","received_events_url":"https://api.github.com/users/chrisgarrity/received_events","type":"User","site_admin":false},"labels":[{"id":253891430,"url":"https://api.github.com/repos/LLK/scratch-www/labels/localization","name":"localization","color":"006b75","default":false},{"id":413979920,"url":"https://api.github.com/repos/LLK/scratch-www/labels/needs%20discussion","name":"needs discussion","color":"efefef","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-12-10T19:11:57Z","updated_at":"2016-12-10T19:11:57Z","closed_at":null,"body":"While working on the transifex import I discovered that the scripts that convert po files to intl/* still have a problem if there are duplicate strings in the same l10n file. Some examples:\n\nIn src/l10n.json:\n\"registration.studentUsernameStepTitle\": \"Create a Scratch Account\",\n\"registration.usernameStepTitleScratcher\": \"Create a Scratch Account\",\n\nIn teacherregistration/l10n.json\n\"teacherRegistration.phoneStepTitle\": \"Phone Number\",\n\"teacherRegistration.phoneNumber\": \"Phone Number\",\n\"teacherRegistration.orgStepTitle\": \"Organization\",\n\"teacherRegistration.organization\": \"Organization\",\n\nDo we really want/need to have the same string defined multiple times in the same view?\n\nFortunately, in the long run transifex will handle the same text with different keys just fine. However, right now this causes a bug in the scripts that convert files - the earlier fix I made handles the same string in different l10n files, but not in the same one. \n\n"}}
{"actor":{"display_login":"chrisgarrity"}}