737145 - Catch vidly upload errors and notify users

Status: VERIFIED FIXED

(Websites :: Firefox Flicks, defect)

Description

[Osmose [:osmose, :mkelly]]

The site should detect when vid.ly sends an error notification and mark a video as having an error during upload, along with notifying the user that their upload has failed.

chelsea: More copy! Becaue bug 737136 is handling the youtube/vimeo link issue, I think this email should be a bit more generic and mention common issues beyond just linking directly to a youtube page. Other issues I can think of:

- Submitting a video format that vidly can't handle (suggest a known-working format?)

- Submitting a link that vidly can't reach (other bug should handle this, but just in case?)

Comment 1

[Chelsea Novak [:chelsea]]

Copy!

There was an error in processing your video. Please upload your video from a personal server or tool like Dropbox. If you are using Dropbox, please make sure to set your video to public. 

If you continue to have problems uploading, please contact us at [e-mail]

Thanks!
-The Firefox Flicks Team

Comment 2

[Osmose [:osmose, :mkelly]]

And the email subject?

Comment 3

[Chelsea Novak [:chelsea]]

There was an error uploading your flick

Comment 4

[Chelsea Novak [:chelsea]]

Sorry, 

There was an error uploading your Firefox Flick

Comment 5

[Osmose [:osmose, :mkelly]]

Code is mostly done, but I'm moving this to 1.3 because it contains string changes and didn't make the string freeze from yesterday. It's not a terribly likely scenario anyway, so there's little risk to the live site from putting this off for another release.

Comment 7

[[github robot]]

Commit pushed to develop at https://github.com/mozilla/firefox-flicks

https://github.com/mozilla/firefox-flicks/commit/0fc52e22a2daa039c2ce1e95900947635e9c737d
Update notification code and send out error emails

fix bug 737145

Comment 8

[Stephen Donner [:stephend] Not actively reading bugmail]

Found bug 737660 while testing this.

Comment 9

[Osmose [:osmose, :mkelly]]

Use http://people.mozilla.org/~mkelly/fakevideo.wmv to test this; it's a fake wmv file that will pass the flicks-side tests.

Comment 10

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED with the testcase/file in comment 9.

Comment 11

[Stephen Donner [:stephend] Not actively reading bugmail]

Attachment: Post-fix screenshot