Percentages: Fix showing of correct values (#243)

ivanahuckova · web-flow · commit 73bb560d8a0d · 2024-06-19T15:37:10.000+02:00
* Percentages: Fix showing of correct values

* Add percentunit to spellcheck words
diff --git a/cspell.config.json b/cspell.config.json
@@ -1,5 +1,10 @@
 {
-  "ignorePaths": ["node_modules/**", "pkg/googlesheets/testdata", "dist/**", "mage_output_file.go"],
+  "ignorePaths": [
+    "node_modules/**",
+    "pkg/googlesheets/testdata",
+    "dist/**",
+    "mage_output_file.go"
+  ],
   "words": [
     "araddon",
     "dataproxy",
@@ -15,6 +20,7 @@
     "instancemgmt",
     "Middlewares",
     "patrickmn",
+    "percentunit",
     "raintank",
     "Sdump",
     "stackdriver",
diff --git a/pkg/googlesheets/columndefinition.go b/pkg/googlesheets/columndefinition.go
@@ -133,7 +133,7 @@ func (cd *ColumnDefinition) checkUnit(cellData *sheets.CellData) {
 			}
 		}
 	case "PERCENT":
-		cd.units["percent"] = true
+		cd.units["percentunit"] = true
 	case "CURRENCY":
 		for unit, unitID := range unitMappings {
 			if strings.Contains(cellData.FormattedValue, unit) {

Original file line number	Diff line number	Diff line change
`@@ -133,7 +133,7 @@ func (cd ColumnDefinition) checkUnit(cellData sheets.CellData) {`
`133`	`133`	`}`
`134`	`134`	`}`
`135`	`135`	`case "PERCENT":`
`136`		`- cd.units["percent"] = true`
	`136`	`+ cd.units["percentunit"] = true`
`137`	`137`	`case "CURRENCY":`
`138`	`138`	`for unit, unitID := range unitMappings {`
`139`	`139`	`if strings.Contains(cellData.FormattedValue, unit) {`