fix groupby name property in merge method with groupby option

AtelierCartographie · May 13, 2024 · 47b32a8 · 47b32a8
1 parent a8fe4ef
commit 47b32a8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/merge.js b/src/merge.js
@@ -38,7 +38,7 @@ export function merge(topo, options = {}) {
   const groupby_features = [];
   groupby_map.forEach((value, key) => {
     const features = topoMergeArcs(topo, value);
-    groupby_features.push({ ...features, properties: { groupby: key } });
+    groupby_features.push({ ...features, properties: { [groupby]: key } });
   });
 
   const output = reconstructTopojson(topo, groupby_features, {

diff --git a/src/properties.js b/src/properties.js
@@ -1,8 +1,8 @@
 import { getLayerName } from "./helpers/layers";
 
 /**
- * Manipulate properties of a topojson layer with Arquero
- * Only four arquero verb are available: select, rename, derive and join
+ * Manipulate properties of a topojson layer
+ * Four types of operations are available: select, rename, derive and join.
  *
  * @param {TopoJSON} topo - A valid topojson object
  * @param {Object} options - optional parameters