@zeniai/client-epic-state 5.2.94-betaNB1 → 5.2.95

package/lib/esm/view/invoicing/invoicingQboView/invoicingQboViewSelector.js

@@ -116,15 +116,13 @@ export const getInvoicingQboAccountOptions = createSelector([
  * Normalises an account-type name for comparison. The config's mapping kinds
  * name QuickBooks types the way QuickBooks displays them ("Other Current
  * Asset", "Expense"), while the accounting service uses snake_case and
- * sometimes pluralises ("other_current_assets", "expenses"). Comparing the
- * normalised, de-pluralised forms keeps a picker from silently emptying because
- * of a trailing "s".
+ * sometimes pluralises ("other_current_assets", "expenses"), and QuickBooks
+ * itself sends PascalCase sub types ("SalesTaxPayable"). Comparing the
+ * separator-free, de-pluralised forms keeps a picker from silently emptying
+ * because of a trailing "s", and an exclusion from missing on casing.
  */
 const normalizeAccountTypeName = (value) => {
-    const normalized = value
-        .trim()
-        .toLowerCase()
-        .replace(/[^a-z0-9]+/g, '_');
+    const normalized = value.toLowerCase().replace(/[^a-z0-9]+/g, '');
     // Singularise so "Other Current Liability" meets `other_current_liabilities`
     // and "Expense" meets `expenses`. `-ies` needs its own case: stripping the
     // trailing "s" alone would leave "liabilitie", matching nothing.

package/lib/view/invoicing/invoicingQboView/invoicingQboViewSelector.js

@@ -125,15 +125,13 @@ exports.getInvoicingQboAccountOptions = (0, toolkit_1.createSelector)([
  * Normalises an account-type name for comparison. The config's mapping kinds
  * name QuickBooks types the way QuickBooks displays them ("Other Current
  * Asset", "Expense"), while the accounting service uses snake_case and
- * sometimes pluralises ("other_current_assets", "expenses"). Comparing the
- * normalised, de-pluralised forms keeps a picker from silently emptying because
- * of a trailing "s".
+ * sometimes pluralises ("other_current_assets", "expenses"), and QuickBooks
+ * itself sends PascalCase sub types ("SalesTaxPayable"). Comparing the
+ * separator-free, de-pluralised forms keeps a picker from silently emptying
+ * because of a trailing "s", and an exclusion from missing on casing.
  */
 const normalizeAccountTypeName = (value) => {
-    const normalized = value
-        .trim()
-        .toLowerCase()
-        .replace(/[^a-z0-9]+/g, '_');
+    const normalized = value.toLowerCase().replace(/[^a-z0-9]+/g, '');
     // Singularise so "Other Current Liability" meets `other_current_liabilities`
     // and "Expense" meets `expenses`. `-ies` needs its own case: stripping the
     // trailing "s" alone would leave "liabilitie", matching nothing.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zeniai/client-epic-state",
-  "version": "5.2.94-betaNB1",
+  "version": "5.2.95",
   "description": "Shared module between Web & Mobile containing required abstractions for state management, async network communication. ",
   "main": "lib/index.js",
   "module": "lib/esm/index.js",