)]}'
{"docs/translation.rst":[{"author":{"_account_id":7509,"name":"Jiri Tomasek","email":"jtomasek@redhat.com","username":"jtomasek"},"change_message_id":"0f7f6962af5afc62398b98f8d0de4f365feb2471","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    npm run po2json -- ./i18n/ja.po -o ./i18n/locales/ja.json"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"Adding a new language"},{"line_number":42,"context_line":"---------------------"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"The languages are defined and activated in 2 places. Additionally, the"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7ffa3b31_1986a014","side":"PARENT","line":41,"updated":"2017-04-20 15:08:52.000000000","message":"This section should maybe stay here if adding additional language requires src/constants/I18n.js change?","commit_id":"fc3983baf76493ee27fad06e7b772760e21fc7b8"},{"author":{"_account_id":20970,"name":"Honza Pokorny","email":"honza@redhat.com","username":"hpokorny"},"change_message_id":"76e8f432e5c70b9ffd9bcbc811cdbef1fd981d42","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    npm run po2json -- ./i18n/ja.po -o ./i18n/locales/ja.json"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"Adding a new language"},{"line_number":42,"context_line":"---------------------"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"The languages are defined and activated in 2 places. Additionally, the"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7ffa3b31_0829a4e4","side":"PARENT","line":41,"in_reply_to":"7ffa3b31_1986a014","updated":"2017-04-20 16:52:28.000000000","message":"Done","commit_id":"fc3983baf76493ee27fad06e7b772760e21fc7b8"}],"src/js/components/i18n/I18nProvider.js":[{"author":{"_account_id":7509,"name":"Jiri Tomasek","email":"jtomasek@redhat.com","username":"jtomasek"},"change_message_id":"0f7f6962af5afc62398b98f8d0de4f365feb2471","unresolved":false,"context_lines":[{"line_number":3,"context_line":"import React from \u0027react\u0027;"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"import I18nActions from \u0027../../actions/I18nActions\u0027;"},{"line_number":6,"context_line":"import { MESSAGES, LOCALE_DATA } from \u0027./messages\u0027;"},{"line_number":7,"context_line":"import { getLanguage, getMessages } from \u0027../../selectors/i18n\u0027;"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"class I18nProvider extends React.Component {"}],"source_content_type":"text/javascript","patch_set":3,"id":"7ffa3b31_195ea075","line":6,"updated":"2017-04-20 15:08:52.000000000","message":"can you please add comment which clarifies that messages.js file is generated by I18nWebpackPlugin?","commit_id":"684f81952d0011eb72f7587d4b6cdad751b96136"},{"author":{"_account_id":20970,"name":"Honza Pokorny","email":"honza@redhat.com","username":"hpokorny"},"change_message_id":"76e8f432e5c70b9ffd9bcbc811cdbef1fd981d42","unresolved":false,"context_lines":[{"line_number":3,"context_line":"import React from \u0027react\u0027;"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"import I18nActions from \u0027../../actions/I18nActions\u0027;"},{"line_number":6,"context_line":"import { MESSAGES, LOCALE_DATA } from \u0027./messages\u0027;"},{"line_number":7,"context_line":"import { getLanguage, getMessages } from \u0027../../selectors/i18n\u0027;"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"class I18nProvider extends React.Component {"}],"source_content_type":"text/javascript","patch_set":3,"id":"7ffa3b31_ad8c02d7","line":6,"in_reply_to":"7ffa3b31_195ea075","updated":"2017-04-20 16:52:28.000000000","message":"Done","commit_id":"684f81952d0011eb72f7587d4b6cdad751b96136"}],"src/js/components/i18n/languageNames.js":[{"author":{"_account_id":4978,"name":"Julie Pichon","email":"jpichon@redhat.com","username":"jpichon"},"change_message_id":"f3b4b69dc02bb727d6febbc17704bffadcead749","unresolved":false,"context_lines":[{"line_number":7,"context_line":"  \u0027ja\u0027: \u0027Japanese\u0027,"},{"line_number":8,"context_line":"  \u0027ko-KR\u0027: \u0027Korean\u0027,"},{"line_number":9,"context_line":"  \u0027zh-CN\u0027: \u0027Simplified Chinese\u0027"},{"line_number":10,"context_line":"};"}],"source_content_type":"text/javascript","patch_set":2,"id":"7ffa3b31_87be1e63","line":10,"updated":"2017-04-18 10:50:50.000000000","message":"It looks like this would still need to be updated when we get a new language, then?\n\nApparently \"only\" 52 languages are available on Zanata at the moment, perhaps we might as well store the whole list here and be done with it?","commit_id":"fc1c14119fa0b29b303a25a775645e52ab3d2270"},{"author":{"_account_id":4978,"name":"Julie Pichon","email":"jpichon@redhat.com","username":"jpichon"},"change_message_id":"778963a44e2b56c8bff3bd0259fd0d4e5af2e1a4","unresolved":false,"context_lines":[{"line_number":7,"context_line":"  \u0027ja\u0027: \u0027Japanese\u0027,"},{"line_number":8,"context_line":"  \u0027ko-KR\u0027: \u0027Korean\u0027,"},{"line_number":9,"context_line":"  \u0027zh-CN\u0027: \u0027Simplified Chinese\u0027"},{"line_number":10,"context_line":"};"}],"source_content_type":"text/javascript","patch_set":2,"id":"7ffa3b31_93a780a8","line":10,"in_reply_to":"7ffa3b31_137370b3","updated":"2017-04-18 12:22:37.000000000","message":"Does it include the name in the language itself? That\u0027d be a fair reason to switch IMO. (There was even a bug open about this once upon a time, bug 1671970)","commit_id":"fc1c14119fa0b29b303a25a775645e52ab3d2270"},{"author":{"_account_id":20970,"name":"Honza Pokorny","email":"honza@redhat.com","username":"hpokorny"},"change_message_id":"4679d0e15cf272129e8f76173c288d2e7c06f858","unresolved":false,"context_lines":[{"line_number":7,"context_line":"  \u0027ja\u0027: \u0027Japanese\u0027,"},{"line_number":8,"context_line":"  \u0027ko-KR\u0027: \u0027Korean\u0027,"},{"line_number":9,"context_line":"  \u0027zh-CN\u0027: \u0027Simplified Chinese\u0027"},{"line_number":10,"context_line":"};"}],"source_content_type":"text/javascript","patch_set":2,"id":"7ffa3b31_137370b3","line":10,"in_reply_to":"7ffa3b31_87be1e63","updated":"2017-04-18 12:10:46.000000000","message":"Yes, this is the only thing.  Unfortunately, the locale data provided by react-intl or Zanata doesn\u0027t include the English names for the various locales.","commit_id":"fc1c14119fa0b29b303a25a775645e52ab3d2270"},{"author":{"_account_id":7509,"name":"Jiri Tomasek","email":"jtomasek@redhat.com","username":"jtomasek"},"change_message_id":"0f7f6962af5afc62398b98f8d0de4f365feb2471","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7ffa3b31_b9bc4c06","updated":"2017-04-20 15:08:52.000000000","message":"This should IMHO live in src/constants/I18n.js","commit_id":"684f81952d0011eb72f7587d4b6cdad751b96136"},{"author":{"_account_id":20970,"name":"Honza Pokorny","email":"honza@redhat.com","username":"hpokorny"},"change_message_id":"76e8f432e5c70b9ffd9bcbc811cdbef1fd981d42","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7ffa3b31_6dc4baf5","in_reply_to":"7ffa3b31_b9bc4c06","updated":"2017-04-20 16:52:28.000000000","message":"Done","commit_id":"684f81952d0011eb72f7587d4b6cdad751b96136"}],"src/js/plugins/i18n.js":[{"author":{"_account_id":4978,"name":"Julie Pichon","email":"jpichon@redhat.com","username":"jpichon"},"change_message_id":"f3b4b69dc02bb727d6febbc17704bffadcead749","unresolved":false,"context_lines":[{"line_number":73,"context_line":""},{"line_number":74,"context_line":"  }"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"}"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"module.exports \u003d I18nPlugin;"}],"source_content_type":"text/javascript","patch_set":2,"id":"7ffa3b31_07e0d195","line":76,"updated":"2017-04-18 10:50:50.000000000","message":"Writing a file server-side on the fly sounds pretty terrifying.","commit_id":"fc1c14119fa0b29b303a25a775645e52ab3d2270"},{"author":{"_account_id":20970,"name":"Honza Pokorny","email":"honza@redhat.com","username":"hpokorny"},"change_message_id":"4679d0e15cf272129e8f76173c288d2e7c06f858","unresolved":false,"context_lines":[{"line_number":73,"context_line":""},{"line_number":74,"context_line":"  }"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"}"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"module.exports \u003d I18nPlugin;"}],"source_content_type":"text/javascript","patch_set":2,"id":"7ffa3b31_f3400438","line":76,"in_reply_to":"7ffa3b31_07e0d195","updated":"2017-04-18 12:10:46.000000000","message":"This is done at build time.  The dist artifact will include this file.  It\u0027s no different than compiling our es6 code to normal javascript.  This is a webpack plugin, and the code doesn\u0027t run when the application is run in production.","commit_id":"fc1c14119fa0b29b303a25a775645e52ab3d2270"},{"author":{"_account_id":4978,"name":"Julie Pichon","email":"jpichon@redhat.com","username":"jpichon"},"change_message_id":"778963a44e2b56c8bff3bd0259fd0d4e5af2e1a4","unresolved":false,"context_lines":[{"line_number":73,"context_line":""},{"line_number":74,"context_line":"  }"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"}"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"module.exports \u003d I18nPlugin;"}],"source_content_type":"text/javascript","patch_set":2,"id":"7ffa3b31_737994fa","line":76,"in_reply_to":"7ffa3b31_f3400438","updated":"2017-04-18 12:22:37.000000000","message":"Ah, I see! So that means if there was an issue with the generated file it\u0027d be caught early at package building time, that\u0027s cool. Thanks for the explanation!","commit_id":"fc1c14119fa0b29b303a25a775645e52ab3d2270"},{"author":{"_account_id":7509,"name":"Jiri Tomasek","email":"jtomasek@redhat.com","username":"jtomasek"},"change_message_id":"c1f865f672ff23fbad73a850d3ccffdbfb35eb26","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7ffa3b31_fc74026a","updated":"2017-04-20 15:17:44.000000000","message":"If a webpack plugin is only thing that limits the nodejs version used, can we consider using es5 here? Limiting a nodejs version may bring problems for other distros in case they want to package tripleo-ui.","commit_id":"684f81952d0011eb72f7587d4b6cdad751b96136"},{"author":{"_account_id":20970,"name":"Honza Pokorny","email":"honza@redhat.com","username":"hpokorny"},"change_message_id":"76e8f432e5c70b9ffd9bcbc811cdbef1fd981d42","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7ffa3b31_a8c3d056","in_reply_to":"7ffa3b31_fc74026a","updated":"2017-04-20 16:52:28.000000000","message":"Porting this file to es5 to get it merged quickly sounds like a good plan.  Supporting outdated platforms doesn\u0027t.","commit_id":"684f81952d0011eb72f7587d4b6cdad751b96136"},{"author":{"_account_id":7509,"name":"Jiri Tomasek","email":"jtomasek@redhat.com","username":"jtomasek"},"change_message_id":"0f7f6962af5afc62398b98f8d0de4f365feb2471","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"const fs \u003d require(\u0027fs\u0027);"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"class I18nPlugin {"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"  constructor(options) {"},{"line_number":31,"context_line":"    this.options \u003d options;"}],"source_content_type":"text/javascript","patch_set":3,"id":"7ffa3b31_19ad809a","line":28,"updated":"2017-04-20 15:08:52.000000000","message":"nit: you can\n\nexport default class I18nPlugin\n\ninstead of module.exports \u003d I18nPlugin;\nbelow","commit_id":"684f81952d0011eb72f7587d4b6cdad751b96136"},{"author":{"_account_id":20970,"name":"Honza Pokorny","email":"honza@redhat.com","username":"hpokorny"},"change_message_id":"76e8f432e5c70b9ffd9bcbc811cdbef1fd981d42","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"const fs \u003d require(\u0027fs\u0027);"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"class I18nPlugin {"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"  constructor(options) {"},{"line_number":31,"context_line":"    this.options \u003d options;"}],"source_content_type":"text/javascript","patch_set":3,"id":"7ffa3b31_084084a4","line":28,"in_reply_to":"7ffa3b31_19ad809a","updated":"2017-04-20 16:52:28.000000000","message":"Done","commit_id":"684f81952d0011eb72f7587d4b6cdad751b96136"}]}