512817 - [1.1.6 & 1.7.1] Upcoming Events

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Developer Pages, defect, P1)

Description

[Justin Scott [:fligtar]]

Attachment: 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.

Comment 1

[Ryan Doherty (:rdoherty)]

fyi, I got ahead of myself and created the html+css for this module for bug 512773

Comment 2

[Scott McCammon [:mccammos]]

Attachment: v1

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  ?

Comment 3

[Wil Clouser [:clouserw]]

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?!

Comment 4

[Scott McCammon [:mccammos]]

Attachment: v2

This patch is fresh as home-baked peach pie! (and it fixes some double-escaping issues found in v1)

Comment 5

[Justin Scott [:fligtar]]

To clear up confusion, the future congregation places will be known as the Editors' Lounge and the Admins' Pad.

Comment 6

[Wil Clouser [:clouserw]]

Comment on attachment 400430 [details] [diff] [review]
v2

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

Comment 7

[Scott McCammon [:mccammos]]

Fixed in r51448, migration in r51449.

Comment 8

[Milos Dinic [:Milos]]

Verified on https://preview.addons.mozilla.org/en-US/firefox/developers and https://preview.addons.mozilla.org/en-US/admin/developershub