Video Detail Page: Add Voting

VERIFIED FIXED

Status

Websites
Firefox Flicks
VERIFIED FIXED
5 years ago
4 years ago

People

(Reporter: bensternthal, Unassigned)

Tracking

Details

(Reporter)

Description

5 years ago
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
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

5 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

5 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

5 years ago
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
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.