Problem/Motivation

Digests should have single button enable / disable operations on the list builder page.

This comes from issue #3167584: Some UI requests.

User interface changes

The list builder page will get new operations for digests. No worries here.

API changes

Adds new routes, no issues with a route rebuild.

CommentFileSizeAuthor
#2 3181409-2.patch6.04 KBderekcresswell

Comments

DerekCresswell created an issue. See original summary.

derekcresswell’s picture

derekcresswell commented
StatusFileSize
new 3181409-2.patch6.04 KB

Simple, just add a controller and routes to the new operations, the list builder base adds the new links automatically.

Status: Needs review » Needs work

The last submitted patch, 2: 3181409-2.patch, failed testing. View results

derekcresswell’s picture

derekcresswell commented

This appears to be the same issue as #3166976: Intermittent test failures with DateTime. The situation seems familiar, later in the night on a sunday / monday. Gives creedence to a logical error or perhaps cron expression skipping dates. I still don't have enough info to figure this out sadly.

derekcresswell’s picture

derekcresswell commented
Status: Needs work » Needs review

Yeah that was the #3166976: Intermittent test failures with DateTime again.

derekcresswell’s picture

derekcresswell commented
Parent issue: » #3166611: [Meta] Release version 1.0.0
joshmiller’s picture

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

Love the tests.

  • DerekCresswell committed cdf1fb4 on 1.0.x 
    Issue #3181409 by DerekCresswell: Add enable / disable operations to...
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.