Skip to content

Commit

Permalink
Merge pull request #2495 from NDLANO/link-to-videoid
Browse files Browse the repository at this point in the history
Do not add t= to videolink
  • Loading branch information
gunnarvelle authored Oct 9, 2024
2 parents 66e5bcc + 548918a commit 2c08207
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/components/SlateEditor/plugins/video/SlateVideo.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,7 @@ const SlateVideo = ({ attributes, element, editor, children }: Props) => {
variant="secondary"
title={t("form.video.brightcove")}
aria-label={t("form.video.brightcove")}
to={`https://studio.brightcove.com/products/videocloud/media/videos/${embed.embedData.videoid}`}
to={`https://studio.brightcove.com/products/videocloud/media/videos/${embed.embedData.videoid.split("&t=")[0]}`}
size="small"
>
<Link />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ const EditLearningResource = ({ isNewlyCreated }: Props) => {
}
const newLanguage = isNewArticleLanguage(selectedLanguage, article);
return (
<ContentTypeProvider value={getContentTypeFromResourceTypes(taxonomyQuery.data?.[0].resourceTypes ?? [])}>
<ContentTypeProvider value={getContentTypeFromResourceTypes(taxonomyQuery.data?.[0]?.resourceTypes ?? [])}>
<HelmetWithTracker title={`${article.title?.title} ${t("htmlTitles.titleTemplate")}`} />
<LearningResourceForm
articleLanguage={selectedLanguage}
Expand Down

0 comments on commit 2c08207

Please sign in to comment.