google-spreadsheet-translation-sync 1.3.8 → 1.3.10

package/README.md

@@ -20,8 +20,15 @@ gsTransSync.exportToSpreadsheet(['de.json', 'en.json'], options, callback);
 gsTransSync.importFromSpreadsheet('your/folder', { spreadsheetId: 'xxxxxxx'}, function () {
     // done :)
 });
-
 ```
+
+On the Google spreadsheets side, this needs to have the following structure:
+
+|key|default|de|en|... any other locale|<locale> # <comment>|
+|---|---|---|---|---|---|
+|test.something|Test Schlüssel|Test Key|****||
+
+Please not the line `<locale> # <comment>` - `# <comment>` will be ignored in uploads and downloads, so you can put some information for your translators into the google doc (They will do it anyway, so this way you don't need to remove the comment before import).
     
 ### Options   
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "google-spreadsheet-translation-sync",
-  "version": "1.3.8",
+  "version": "1.3.10",
   "description": "A plugin to read and write i18n translationsfrom and to google spreadsheets ",
   "main": "index.js",
   "scripts": {

package/src/export-to-spreadsheet.js

@@ -160,7 +160,9 @@ module.exports = function (translationFiles, options, callback) {
 
                         // this is the header row
                         if (cell.value) {
-                          headerIndexMap[cellIndex] = keyIndexMap[cell.value];
+                          // clear comments from the header name
+                          let cleanLocale = cell.value.split('#')[0].trim();
+                          headerIndexMap[cellIndex] = keyIndexMap[cleanLocale];
                           maxIndex = cellIndex
                         } else {
                           // this looks like an initial upload, let's do it

package/src/import-from-spreadsheet.js

@@ -42,6 +42,9 @@ module.exports = function (translationRootFolder, options, callback) {
 
                                 if (rowIndex === 0) {
                                     if (val) {
+                                        // clear comments from the header
+                                        val = val.split('#')[0].trim();
+
                                         headers[cellIndex] = val;
                                         if (cellIndex > keyCellIndex) {
                                             translationData[val] = {};

package/test/test.js

@@ -9,10 +9,10 @@ const accessData = require('./data/google-test-access.json');
 const tmp = require('tmp');
 const async = require('async');
 
-const testSheetId = '1ZJK1G_3wrEo9lnu1FenjOzSy3uoAi-RLWbph1cI6DWI';
+const testSheetId = '1ZJK1G_3wrEo9lnu1FenjOzSy3uoAi-RLWbph1cI6DWI'; // https://docs.google.com/spreadsheets/d/1ZJK1G_3wrEo9lnu1FenjOzSy3uoAi-RLWbph1cI6DWI/edit#gid=0
 const testWorksheetId = '1209225803';
 const testSheetId_gettext = '1CRvX4TCxUGCcs_MtKC5BdEViHYzYzLXdqtbuVaAXfKc';
-const testSheetId_properties = '1Z0Mpbf6lgdGiuiHlpb9DENVfKxkxSRwcfQEDYrgokEE';
+const testSheetId_properties = '1Z0Mpbf6lgdGiuiHlpb9DENVfKxkxSRwcfQEDYrgokEE'; // https://docs.google.com/spreadsheets/d/1Z0Mpbf6lgdGiuiHlpb9DENVfKxkxSRwcfQEDYrgokEE/edit#gid=0
 const testSheetId_yaml = '1Ml3jLK6RgbPQ4e7XZpILvB-XWMeziHpwdxS8r4AYNtE';
 const timeout = 20000;
 
@@ -202,7 +202,7 @@ const tests = [
                 }).then(function (rows) {
                   expect(rows).to.have.lengthOf((csvData.length - 1) * 2);
                   expect(rows[0][options.keyId]).to.equal(csvData[1][0]);
-                  expect(rows[0].pl).to.equal(csvData[1][5]);
+                  expect(rows[0]['pl # with comment']).to.equal(csvData[1][5]);
                   expect(rows[0].default).to.equal(csvData[1][1]);
                   expect(rows[0].hu).to.equal('Elfogadom');
                   expect(rows[0].namespace).to.equal(namespaces[0]);
@@ -281,7 +281,7 @@ const tests = [
             }).then(function (rows) {
               expect(rows).to.have.lengthOf(csvData.length - 1);
               expect(rows[0][options.keyId]).to.equal(csvData[1][0]);
-              expect(rows[0].pl).to.equal(csvData[1][5]);
+              expect(rows[0]['pl # with comment']).to.equal(csvData[1][5]);
               expect(rows[0].default).to.equal(csvData[1][1]);
               expect(rows[0].hu).to.equal('Elfogadom'); // this was not part of the upload and should not be overwrittem
               expect(rows[1].default).to.equal(csvData[2][1]);
@@ -462,7 +462,7 @@ const tests = [
               }).then(function (rows) {
                 expect(rows).to.have.lengthOf(csvData.length - 1);
                 expect(rows[0][options.keyId]).to.equal(csvData[1][0]);
-                expect(rows[0].pl).to.equal(csvData[1][5]);
+                expect(rows[0]['pl # with comment']).to.equal(csvData[1][5]);
                 expect(rows[0].default).to.equal(csvData[1][1]);
                 expect(rows[0].hu).to.equal('Elfogadom'); // this was not part of the upload and should not be overwrittem
                 expect(rows[1].default).to.equal(csvData[2][1]);
@@ -575,7 +575,7 @@ const tests = [
               }).then(function (rows) {
                 expect(rows).to.have.lengthOf(csvData.length - 1);
                 expect(rows[0][options.keyId]).to.equal(csvData[1][0]);
-                expect(rows[0].pl).to.equal(csvData[1][5]);
+                expect(rows[0]['pl # with comment']).to.equal(csvData[1][5]);
                 expect(rows[0].default).to.equal(csvData[1][1]);
                 expect(rows[0].hu).to.equal('Elfogadom'); // this was not part of the upload and should not be overwrittem
                 expect(rows[1].default).to.equal(csvData[2][1]);