bugzilla.mozilla.org has resumed normal operation. Attachments prior to 2014 will be unavailable for a few days. This is tracked in Bug 1475801.
Please report any other irregularities here.

staff-only videos get featured for logged-in contributors

RESOLVED FIXED

Status

Webtools
Air Mozilla
P2
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: philipp, Assigned: peterbe)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

5 years ago
when i go to https://air.mozilla.org/ and log in with the persona tied to my mozillian contributor profile, the featured videos section on the right side of the page changes, but then it also includes recordings that are marked as staff-only, which cannot be accessed subsequently.

since this is in some cases rather frustrating, it's probably better to hide the interesting stuff from us, that we cannot get to :))

Updated

5 years ago
Assignee: nobody → peterbe
(Assignee)

Updated

5 years ago
Component: Other → Air Mozilla
Product: Air Mozilla → Webtools
Target Milestone: --- → Future
Version: unspecified → 0.1
(Assignee)

Updated

5 years ago
Target Milestone: Future → ---
Version: 0.1 → other
(Assignee)

Comment 1

5 years ago
Thank you! That's regression I think. I know there is code to prevent that from happening. Clearly it's buggy. 

Interestingly, we're completely changing how "Featured Videos" works. Instead it's going to be called "Trending Events" and it's sorting will be based on hits and age and it's almost the same algorithm used by Hacker News home page. I'll try to assert that it works accordingly (with regards to privacy) in the new functionality.
(Assignee)

Updated

5 years ago
Status: UNCONFIRMED → NEW
Ever confirmed: true
Priority: -- → P2

Comment 2

5 years ago
Commit pushed to master at https://github.com/mozilla/airmozilla

https://github.com/mozilla/airmozilla/commit/6073ac485a920ddee24d1e03cfd9efaef958484d
fixes bug 964388 - staff-only videos get featured for logged-in contributors

Updated

5 years ago
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Assignee)

Comment 3

5 years ago
Thanks for filing this philipp. It was easy to see where the mistake was and I was able to prove it with a new test specifically for this. 

Will probably go into production next week.
You need to log in before you can comment on or make changes to this bug.