address review feedback; add ability to enable/disable canonical story URL

Author: kabeaty

README.md

@@ -16,15 +16,15 @@ Then you will need to:
 
 ## Story settings
 
-The plugin uses the canonical URL for a Wordpress post for the Coral Story URL.
+The plugin supports enabling canonical URLs for a Wordpress post to be passed through to the Coral stream embed as the story URL.
 
 The plugin also supports setting a story mode and sending it through to the Coral stream embed. To use, you can add a custom field in Wordpress called `coralStoryMode` and then set it to a valid story mode.
 
 ## HTTPS and Dev Mode
 
 Your site must be served over `https` in order to integrate with Coral **unless** Coral is set to dev mode.
 
-If you're installing Coral with Docker, you can do that by adding `NODE_ENV=dev` to the environment variables in your [`docker-compose.yml`](https://docs.coralproject.net/environment-variables#node_env). Otherwise, any method of setting `process.env.NODE_ENV = 'dev'` will do the trick.
+If you're installing Coral with Docker, you can do that by adding `NODE_ENV=development` to the environment variables in your [`docker-compose.yml`](https://docs.coralproject.net/environment-variables#node_env). Otherwise, any method of setting `NODE_ENV=development` will do the trick.
 
 ## Theme usage
 
@@ -58,4 +58,4 @@ Coral version <= `v3.9.1` use plugin version `v0.0.6`
 
 Coral version >= `4.0.0` use plugin version `v0.1.0`
 
-Coral version >= `5.0.0` use plugin version `v0.2.1`
+Coral version >= `5.0.0` use plugin version `v1.0.0`

inc/class-talk-settings-page.php

@@ -127,6 +127,15 @@ function() {
 			'about-talk'
 		);
 		register_setting( 'talk-settings', 'coral_custom_scroll_container' );
+
+		add_settings_field(
+			'coral_enable_canonical_story_urls',
+			__( 'Enable canonical story urls', 'coral-project-talk' ),
+			array( $this, 'render_enable_canonical_story_urls_field' ),
+			'talk-settings',
+			'about-talk'
+		);
+		register_setting( 'talk-settings', 'coral_enable_canonical_story_urls' );
 	}
 
 	/**
@@ -269,6 +278,42 @@ public function render_disable_default_fonts_field() {
 				id="coral_disable_default_fonts"
 				type="select"
 		>
+		<option value="false"
+				<?php 
+					if ($default_fonts === "false")
+						echo "selected=\"selected\""
+				?>
+			>
+				false
+			</option>
+		<option value="true"
+				<?php 
+					if ($default_fonts === "true")
+						echo "selected=\"selected\""
+				?>
+			>
+				true
+			</option>
+	</select>
+		<p class="description">Disable default fonts will turn off font-face loading of Coral's default fonts.</p>
+		<?php
+	}
+
+	/**
+	 * Prints input field for disable default fonts setting.
+	 *
+	 * @since 1.0.0
+	 */
+	public function render_enable_canonical_story_urls_field() {
+		$default_fonts = esc_attr( get_option( 'coral_enable_canonical_story_urls' ) )
+		?>
+		<select
+				style="width: 600px; height: 40px;"
+				name="coral_enable_canonical_story_urls"
+				placeholder=""
+				id="coral_enable_canonical_story_urls"
+				type="select"
+		>
 		<option value="true"
 				<?php 
 					if ($default_fonts === "true")
@@ -286,7 +331,7 @@ public function render_disable_default_fonts_field() {
 				false
 			</option>
 	</select>
-		<p class="description">Disable default fonts will turn off font-face loading of Coral's default fonts.</p>
+		<p class="description">When enabled, the canonical story URL will be passed through to the Coral stream embed.</p>
 		<?php
 	}
 

inc/comments-template.php

@@ -19,6 +19,7 @@
 $coral_container_class_name = get_option( 'coral_container_class_name', null);
 $coral_disable_default_fonts = get_option( 'coral_disable_default_fonts' );
 $coral_custom_scroll_container = get_option( 'coral_custom_scroll_container', null );
+$enable_canonical_url = get_option( 'coral_enable_canonical_story_urls' );
 $canonical_url = wp_get_canonical_url();
 $storyMode = $post->coralStoryMode;
 
@@ -34,6 +35,7 @@
 	(function() {
 		var d = document, s = d.createElement('script');
 		const disableDefaultFonts = "<?php echo esc_attr( $coral_disable_default_fonts ); ?>" === "true";
+		const storyURL = "<?php echo esc_attr( $enable_canonical_url ); ?>" === "true" ? "<?php echo esc_url( $canonical_url ); ?>" : undefined;
 		s.src = "<?php echo esc_url( $talk_url . '/assets/js/embed.js' ); ?>"
 		s.onload = function() {
 			const customScrollContainer = document.getElementById("<?php echo esc_attr( $coral_custom_scroll_container ); ?>") || undefined;
@@ -46,7 +48,7 @@
 				customFontsCSSURL: "<?php echo esc_url( $coral_custom_fonts_css_url ); ?>",
 				disableDefaultFonts: disableDefaultFonts,
 				customScrollContainer: customScrollContainer,
-				storyURL: "<?php echo esc_url( $canonical_url ); ?>",
+				storyURL: storyURL,
 				storyMode: "<?php echo esc_attr( $storyMode ); ?>",
 			});
 		};