Closed
Bug 886365
Opened 11 years ago
Closed 11 years ago
Video Detail Page: Add Voting
Categories
(Websites :: Firefox Flicks, defect)
Websites
Firefox Flicks
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: bensternthal, Unassigned)
References
Details
This bug captures adding voting to the site and to the video detail page as described in wireframe page 5,6. - Voting Requires Login - Share is Added To Voting Flow - Voting State Displayed Via Icon On Page
Comment 1•11 years ago
|
||
Craig: FYI for the frontend portion of this, we also need to determine how to message the user in the event of an error. Given that there are a few ways around the site this can happen, I'd recommend we figure out a site-wide way to show that an error has occurred, perhaps a red bar at the top of the page that slides down? We can come up with a generic message with recommendations that the user try again and then email us.
Comment 2•11 years ago
|
||
Commits pushed to master at https://github.com/mozilla/firefox-flicks https://github.com/mozilla/firefox-flicks/commit/b2ab5a45c5cc85ee44817da5e45b499f885fac7a Bug 886365: Add basic voting. https://github.com/mozilla/firefox-flicks/commit/b93209ca332b45072aabdc260af040e466e39b96 Bug 886365: Prompt for login prior to voting if necessary.
Comment 3•11 years ago
|
||
Commit pushed to master at https://github.com/mozilla/firefox-flicks https://github.com/mozilla/firefox-flicks/commit/ddc3e53da4a316a35be36a7ec9a8312395f09f62 Fix bug 886365: Hook up the new frontend voting markup to the JS.
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 4•11 years ago
|
||
Verified login, share options, and vote status on the video page in dev.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•