Merge pull request #777 from egargan-ft/ELES-1237-fix-no-highlights-h…

…andling

Fix automatic 'setIncludeHighlights' + handle errors from highlights token request (ELES-1237)

components/x-gift-article/src/SharingOptionsToggler.jsx

@@ -18,7 +18,6 @@ export const SharingOptionsToggler = (props) => {
 				actions.showAdvancedSharingOptions()
 			} else {
 				actions.hideNonSubscriberSharingOptions(event)
-				actions.setIncludeHighlights(event.target.checked)
 			}
 			return
 		}

components/x-gift-article/src/lib/highlightsApi.js

@@ -38,6 +38,11 @@ export default class HighlightsApiClient {
 		)
 
 		const response = await fetch(url, options)
+
+		if (!response.ok) {
+			throw new Error(`failed to fetch ${url}, received ${response.status}`)
+		}
+
 		const responseJSON = await response.json()
 		return responseJSON
 	}
@@ -62,6 +67,7 @@ export default class HighlightsApiClient {
 			if (!includeHighlights) {
 				return {}
 			}
+
 			return await this.fetchJson('/create-token', {
 				method: 'POST',
 				body: JSON.stringify({ articleId })