748035 - Broken path to .mp4 version of video on Get Involved page

Status: RESOLVED INVALID

(www.mozilla.org :: General, defect)

Description

[Terry R.]

From here:  http://www.mozilla.org/en-US/contribute/

The source contains two forward slashes before "http" which need to be removed.

<source src="//http://videos-cdn.mozilla.net/brand/Mozilla_Firefox_Manifesto_v0.2_640.mp4" type="video/mp4" />

Comment 1

[David Boswell]

I just made this edit to fix the path and submitted the following pull request

https://github.com/davidwboswell/bedrock/commit/d97eb5f574c3740727048f8f444f6a7195ce18a9

CC'ing Mike, James and Steven for next steps.

Comment 2

[David Boswell]

FWIW, I tried testing and couldn't get the video to play on Safari -- I imagine that's related to this.

Comment 3

[James Long (:jlongster)]

Looks good, I actually remember fixing this before but it might have gotten lost on a branch somewhere? Weird.

David, can you submit a pull request with that fix?

Comment 4

[James Long (:jlongster)]

Oh, I see you already did.

Comment 5

[[github robot]]

Commits pushed to dev at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/d97eb5f574c3740727048f8f444f6a7195ce18a9
Fixing path to .mp4 version of video -- bug 748035

https://github.com/mozilla/bedrock/commit/05034c4b0fdb25ce00a38998c442e71de91ddcd2
Merge pull request #69 from davidwboswell/patch-4

Fixing path to .mp4 version of video -- bug 748035

Comment 6

[David Boswell]

(In reply to David Boswell from comment #2)
> FWIW, I tried testing and couldn't get the video to play on Safari -- I
> imagine that's related to this.

To follow up on this, I tried testing the page on stage with Safari and the video still wouldn't load.  Maybe related to another issue.

Comment 7

[Steven Garrity [:sgarrity]]

This video isn't on this page any more.