Another minor issue, we have a platform built on top of Panopoly, and are trying to keep all modules as current as possible, so creating this issue and patch (coming) so I can test, would like to get off as many -dev or older modules as possible…

CommentFileSizeAuthor
#2 panopoly_admin-update_date_popup_authored_module-2936259-2.patch698 bytesbyronveale

Comments

byronveale created an issue. See original summary.

byronveale’s picture

byronveale commented
StatusFileSize
new panopoly_admin-update_date_popup_authored_module-2936259-2.patch698 bytes

And here's the patch (attached).

byronveale’s picture

byronveale commented
Status: Active » Needs review
bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
commented
Status: Needs review » Reviewed & tested by the community

We've had this in prod for weeks w/o issue.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented

Thanks! Here's a test build:

https://travis-ci.org/panopoly/panopoly/builds/376078331

byronveale’s picture

byronveale commented

Repeating myself again, but here's some more low-hanging fruit for you.

  • dsnopek committed bc52128 on 7.x-1.x 
    Update Panopoly Admin for Issue #2936259 by byronveale, bkosborne:...
dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Status: Reviewed & tested by the community » Fixed

Thanks! The tests passed and I did a tiny bit of manual testing, and this seemed fine. Committed!

byronveale’s picture

byronveale commented

Heh, I've really got to make sure I reload pages before I comment on them…

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.