Merge pull request #54 from OneNoteDev/feature/add-clipper-queryparam-to-viewinono

Add query param so OneNote Online knows it was navigated to from the Web Clipper

Author: mttwc

src/scripts/clipperUI/panels/successPanel.tsx

@@ -18,7 +18,8 @@ class SuccessPanelClass extends ComponentBase<{ }, ClipperStateProp> {
 		Clipper.logger.logUserFunnel(Log.Funnel.Label.ViewInWac);
 		let data = this.props.clipperState.oneNoteApiResult.data as OneNoteApi.Page;
 		if (data && data.links && data.links.oneNoteWebUrl && data.links.oneNoteWebUrl.href) {
-			window.open(data.links.oneNoteWebUrl.href, "_blank");
+			let urlWithFromClipperParam = Utils.addUrlQueryValue(data.links.oneNoteWebUrl.href, Constants.Urls.QueryParams.wdFromClipper, "1");
+			window.open(urlWithFromClipperParam, "_blank");
 		} else {
 			Clipper.logger.logFailure(Log.Failure.Label.OnLaunchOneNoteButton, Log.Failure.Type.Unexpected,
 				{ error: "Page created and returned by API is either missing entirely, or missing its links, oneNoteWebUrl, or oneNoteWebUrl.href. Page: " + data });

src/scripts/constants.ts

@@ -318,6 +318,7 @@ export module Constants {
 			export var timeoutInMs = "timeoutInMs";
 			export var url = "url";
 			export var userSessionId = "userSessionId";
+			export var wdFromClipper = "wdfromclipper"; // This naming convention is standard in OneNote Online
 		}
 	}