Merge pull request #258 from OfficeDev/multiword-names

Rick-Kirkham · web-flow · commit 12bd3c877079 · 2024-04-25T16:15:39.000-07:00
Update conversion script to support multiword add-in names
diff --git a/convertToSingleHost.js b/convertToSingleHost.js
@@ -279,7 +279,7 @@ if (projectName) {
   }
 
   // Modify the manifest to include the name and id of the project
-  const cmdLine = `npx office-addin-manifest modify ${manifestPath} -g ${appId} -d ${projectName}`;
+  const cmdLine = `npx office-addin-manifest modify ${manifestPath} -g ${appId} -d "${projectName}"`;
   childProcess.exec(cmdLine, (error, stdout) => {
     if (error) {
       console.error(`Error updating the manifest: ${error}`);

Original file line number	Diff line number	Diff line change
`@@ -279,7 +279,7 @@ if (projectName) {`
`279`	`279`	`}`
`280`	`280`
`281`	`281`	`// Modify the manifest to include the name and id of the project`
`282`		- const cmdLine = `npx office-addin-manifest modify ${manifestPath} -g ${appId} -d ${projectName}`;
	`282`	+ const cmdLine = `npx office-addin-manifest modify ${manifestPath} -g ${appId} -d "${projectName}"`;
`283`	`283`	`childProcess.exec(cmdLine, (error, stdout) => {`
`284`	`284`	`if (error) {`
`285`	`285`	console.error(`Error updating the manifest: ${error}`);