[1.1.6 & 1.7.1] Upcoming Events

VERIFIED FIXED in 5.1

Status

defect
P1
normal
VERIFIED FIXED
10 years ago
3 years ago

People

(Reporter: fligtar, Assigned: smccammon)

Tracking

unspecified

Details

()

Attachments

(2 attachments, 1 obsolete attachment)

Reporter

Description

10 years ago
Posted image mockup
Filing as one bug because they'll be implemented by the same person most likely
at all once, so why require 2 patches?

Sidebar:
* Upcoming events should be displayed in a box in the sidebar with the event name as a link, the date in short form, and the location.
* After the event's date has passed, the event should no longer be displayed
* The list should be ordered with the closest date first.
* If no events are upcoming, the box should be hidden.
* The icon shown in the mockup should not be included.

Admin Tool:
* Events are created, edited, and deleted via an admin tool. Each event has a name, link, date, and location. Events are not localized.
Reporter

Updated

10 years ago
Assignee: nobody → smccammon
fyi, I got ahead of myself and created the html+css for this module for bug 512773
Assignee

Comment 2

10 years ago
Posted patch v1 (obsolete) — Splinter Review
Styles for the calendar can be found in the patch for bug 512773.

What do you think about changing the admin url from:
/admin/developershub/stuff/morestuff

to: /admin/hub/stuff/morestuff  ?
Attachment #400148 - Flags: review?(clouserw)
Attachment #400148 - Flags: review?(clouserw) → review-
Comment on attachment 400148 [details] [diff] [review]
v1

I'm r-ing because it's bitrotted (sorry).  Can you svn up and then repatch?

As far as renaming to /hub/, what happens when we make an admin hub so we can all hang out on it?  Or an editors hub?!
Assignee

Comment 4

10 years ago
Posted patch v2Splinter Review
This patch is fresh as home-baked peach pie! (and it fixes some double-escaping issues found in v1)
Attachment #400148 - Attachment is obsolete: true
Attachment #400430 - Flags: review?(clouserw)
Reporter

Comment 5

10 years ago
To clear up confusion, the future congregation places will be known as the Editors' Lounge and the Admins' Pad.
Attachment #400430 - Flags: review?(clouserw) → review+
Comment on attachment 400430 [details] [diff] [review]
v2

That is fresh!  (still have double escaping on input error, but that's not a blocker)
Assignee

Comment 7

10 years ago
Fixed in r51448, migration in r51449.
Status: NEW → RESOLVED
Closed: 10 years ago
Keywords: push-needed
Resolution: --- → FIXED
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.