Merge pull request #787 from trey-wallis/dev

8.6.3

Former-commit-id: b940e50

Author: decaf-dev

manifest.json

@@ -10,5 +10,5 @@
 		"GitHub": "https://github.com/sponsors/trey-wallis",
 		"Buymeacoffee": "https://www.buymeacoffee.com/treywallis"
 	},
-	"version": "8.6.2"
+	"version": "8.6.3"
 }

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "obsidian-dataloom",
-	"version": "8.6.2",
+	"version": "8.6.3",
 	"description": "Weave together data from diverse sources into a cohesive table view. Inspired by Excel Spreadsheets and Notion.so.",
 	"main": "main.js",
 	"scripts": {

src/react/import-app/state-utils.spec.ts

@@ -211,4 +211,5 @@ describe("addImportData", () => {
 	});
 });
 
+//TODO add tests for spacing
 //TODO add tests for tags, date cell, etc

src/react/import-app/state-utils.ts

@@ -73,7 +73,7 @@ export const addImportData = (
 				let newCell: Cell | null = null;
 				if (match) {
 					const { importColumnIndex } = match;
-					content = importRow[importColumnIndex];
+					content = importRow[importColumnIndex].trim();
 
 					if (type === CellType.TAG) {
 						const { cell, newTags } = createTagCell(
@@ -82,15 +82,15 @@ export const addImportData = (
 							content
 						);
 						newCell = cell;
-						column.tags.push(...newTags);
+						columnTags.push(...newTags);
 					} else if (type === CellType.MULTI_TAG) {
 						const { cell, newTags } = createMultiTagCell(
 							columnTags,
 							columnId,
 							content
 						);
 						newCell = cell;
-						column.tags.push(...newTags);
+						columnTags.push(...newTags);
 					} else if (type === CellType.DATE) {
 						const cell = createDateCell(columnId, content);
 						newCell = cell;
@@ -130,10 +130,19 @@ const createMultiTagCell = (
 	const newTags: Tag[] = [];
 	const tagIds: string[] = [];
 
-	const parsedTags = content.split(",");
+	const parsedTags = content.split(",").map((tag) => tag.trim());
 	if (parsedTags.length !== 0) {
 		parsedTags.forEach((tag) => {
-			const existingTag = columnTags.find((t) => t.content === tag);
+			const existingTag = columnTags.find((t) => {
+				if (columnTags.length === 6) {
+					console.log({
+						content: t.content,
+						tag,
+						eq: t.content === tag,
+					});
+				}
+				return t.content === tag;
+			});
 			if (existingTag) {
 				tagIds.push(existingTag.id);
 			} else {
@@ -147,6 +156,10 @@ const createMultiTagCell = (
 	const cell = createCell(columnId, {
 		tagIds,
 	});
+	console.log({
+		cell,
+		newTags,
+	});
 	return {
 		cell,
 		newTags,
@@ -161,7 +174,7 @@ const createTagCell = (
 	const newTags: Tag[] = [];
 	let tagId: string | null = null;
 
-	const parsedTags = content.split(",");
+	const parsedTags = content.split(",").map((tag) => tag.trim());
 	if (parsedTags.length !== 0) {
 		parsedTags.forEach((tag) => {
 			const existingTag = columnTags.find((t) => t.content === tag);

versions.json

@@ -124,5 +124,6 @@
 	"8.5.0": "1.2.0",
 	"8.6.0": "1.2.0",
 	"8.6.1": "1.2.0",
-	"8.6.2": "1.2.0"
+	"8.6.2": "1.2.0",
+	"8.6.3": "1.2.0"
 }