[GitHub] kristw commented on a change in pull request #6600: Add doc about translation using poedit

[GitBox <gi...@apache.org>]

kristw commented on a change in pull request #6600: Add doc about translation using poedit
URL: https://github.com/apache/incubator-superset/pull/6600#discussion_r246681246
 
 

 ##########
 File path: CONTRIBUTING.md
 ##########
 @@ -446,6 +449,8 @@ If you get errors running `po2json`, you might be running the Ubuntu package wit
 name, rather than the NodeJS package (they have a different format for the arguments). If
 there is a conflict, you may need to update your `PATH` environment variable or fully qualify
 the executable path (e.g. `/usr/local/bin/po2json` instead of `po2json`).
+If you get a lot of`[null,***]`in `messages.json` just delete all the `null,` 
 
 Review comment:
   - Add space around ``[null,***]``
   - full-stop after `null,`
   - Add comma after "For example"
   - Add full-stop after "is incorrect"

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org