If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Derby banners are shown outside the Derby where they are earned

RESOLVED WONTFIX

Status

Mozilla Developer Network
Demo Studio / Dev Derby
P2
normal
RESOLVED WONTFIX
5 years ago
2 years ago

People

(Reporter: openjck, Unassigned)

Tracking

Details

(Whiteboard: [specification][type:bug][specification-comment:1][triaged][might-expire])

(Reporter)

Description

5 years ago
If a demo is a finalist or winner of a Dev Derby, a colored banner appears above that demo's thumbnail. These banner are displayed throughout the Demo Studio, which can be confusing at times.

For example, take a look at the September 2012 entries (https://developer.mozilla.org/en-US/demos/devderby/2012/september). The demo "Meet me there" has a "Second Place" banner, even though (at the time of this writing) it has not won second place in that Derby. The banner is appearing because it won second place in a different Derby.

Things can get pretty confusing on the Demo Studio, too. If you browse through the demos, you can see multiple demos with (for example) a "First Place" banner.

Both of these issues could be solved by only showing banners in the Derby that the banner was won.
(Reporter)

Comment 1

5 years ago
Comment 0 is slightly out of date. "Meet me there" ended up winning third place. The problem is still more-or-less the same, but a little more visible.

What did you do?
================
1. Visit https://developer.mozilla.org/demos/devderby/2012/september

What happened?
==============
"Meet me there" won third place, but actually has a "Second Place" banner because it won second place the month before. As I mention in comment 0, this was a problem even before the winners were announced. At that time, "Meet me there" had a "Second Place" banner even though no winners had been announced.

What should have happened?
==========================
Before the winners were announced, "Meet me there" should not have had a banner on the September 2012 landing page, and a "Second Place" banner in the August landing page. After "Meet me there" was named as the third place winner for September, it should have had only a "Third Place" banner on the September landing page and a "Second Place" banner on the August landing page.

Is there anything else we should know?
======================================
Whiteboard: [specification][type:bug][specification-comment:1]
(Reporter)

Updated

5 years ago
Summary: Do not show Derby banners outside of the Derby where it was earned → Derby banners are shown outside the Derby where they are earned
Part of this seems conflated with re-submitting the same demo for multiple derbies. I thought we locked them and required creation of a new demo for a new derby, if only to freeze the source used and not allow tweaks past the date of the competition?

Also, seems like this could use some design thought as to how to present these banners, especially if we do anticipate multiple honors for the same demo
(Reporter)

Comment 3

5 years ago
The rules do allow a contestant to submit a demo to a second Derby. I always assumed that we wanted to use the same exact submission (the rules mention contacting one of us to do it manually), but I think you make a good point about just asking them to submit a new demo.

Ideally, I would love if demos were tied to Git repositories, and each Derby submission was just the repo at a particular tag. But I will open a separate bug about that later.

Still, might be worth fixing for the existing problem.
(In reply to John Karahalis [:openjck] from comment #3)
> The rules do allow a contestant to submit a demo to a second Derby. I always
> assumed that we wanted to use the same exact submission (the rules mention
> contacting one of us to do it manually), but I think you make a good point
> about just asking them to submit a new demo.

I don't recall the bug #, but I seem to remember we implemented logic to prevent changes to a demo after the finish date of a derby submission. Submitting new demos for new derbies were a part of that discussion, I think

> Ideally, I would love if demos were tied to Git repositories, and each Derby
> submission was just the repo at a particular tag. But I will open a separate
> bug about that later.
> 
> Still, might be worth fixing for the existing problem.

Yeah, that's a different issue, and I don't think we should require all demo authors to use github. Github integration should be a nice addition, but I don't think we should pin functionality on them
Oof, found the bug for reference, and it's ancient:

https://bugzilla.mozilla.org/show_bug.cgi?id=672962#c1

Could go either way with this, I think: Either require fresh submissions for subsequent derbies (technically simpler), or allow refreshes to existing demos & rethink how multiple derby wins are displayed (technically harder, needs some design consideration)

If we allow demos to be repeat submissions to derbies, we probably want to rework the edit freeze to be temporary for the span of the derby judging period so entrants can't continue to tweak
(Reporter)

Comment 6

5 years ago
(In reply to Les Orchard [:lorchard] from comment #4)
> I don't recall the bug #, but I seem to remember we implemented logic to
> prevent changes to a demo after the finish date of a derby submission.
> Submitting new demos for new derbies were a part of that discussion, I think

Yeah. Impossible to change a demo after the Derby ends.
Multiple derby wins on a single demo might be cool, but it might be easier to just track derby wins by contributor instead? And give them some cool little avatar medals or something?
(Reporter)

Updated

5 years ago
Priority: -- → P2
(Reporter)

Comment 8

4 years ago
Still happens, I am almost certain.
OS: Linux → All
Hardware: x86_64 → All
Whiteboard: [specification][type:bug][specification-comment:1] → [specification][type:bug][specification-comment:1][triaged][might-expire]

Updated

4 years ago
Depends on: 999635

Comment 9

2 years ago
Demo Studio is being removed from MDN and archived as of end January 2016
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.