Problem/Motivation

Currently the digest entities use a custom value for dictating if they are enabled or not. This creates some duplicated functionality.

Proposed resolution

Remove the 'enabled' value and functions for it from the digest entity. Switch to utilise the 'status' value.

CommentFileSizeAuthor
#3 3168353-3.patch9.82 KBderekcresswell
#2 3168353-2.patch10.1 KBderekcresswell

Comments

DerekCresswell created an issue. See original summary.

derekcresswell’s picture

derekcresswell commented
Status: Active » Needs review
StatusFileSize
new 3168353-2.patch10.1 KB

Here is a preliminary patch. This removes enabled from the digests.

derekcresswell’s picture

derekcresswell commented
StatusFileSize
new 3168353-3.patch9.82 KB

Updating to match new HEAD of the repo.

joshmiller’s picture

joshmiller
he/him
Primary language English
Location Indianapolis, Indiana, USA
commented
Status: Needs review » Reviewed & tested by the community

  • DerekCresswell committed cef1086 on 1.0.x 
    Issue #3168353 by DerekCresswell: Utilise Config entity 'status' instead...
derekcresswell’s picture

derekcresswell commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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