Totally minor, removes the need for this patch that is present in panopoly_core.

We have a platform built on top of Panopoly, and are trying to keep all modules as current as possible, so creating this issue…

CommentFileSizeAuthor
#2 panopoly_core-update_apps_module-2936030-2.patch571 bytesbyronveale

Comments

byronveale created an issue. See original summary.

byronveale’s picture

byronveale commented
StatusFileSize
new panopoly_core-update_apps_module-2936030-2.patch571 bytes

…and uploading this patch (attached).

byronveale’s picture

byronveale commented
Issue summary: View changes
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 production for weeks without issue.

byronveale’s picture

byronveale commented

so, as I mentioned in the issue summary, totally minor, but if you're committing stuff, figured this would be low-hanging fruit…

  • dsnopek committed ed30304 on 7.x-1.x 
    Update Panopoly Core for Issue #2936030 by byronveale, bkosborne: Update...
dsnopek’s picture

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

Thanks, this one had kind of slipped off my radar! Looking at what's changed in Apps, it's really just adding a patch we're already using, so I think it's safe to do this. Committed!

Status: Fixed » Closed (fixed)

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