Since published doesn't mean anything else yet. There was a todo for this, and Joe and I had agreed that it was probably better to use published. I had some test fails locally but they seemed spurious, so Tim, let me know if this breaks something for you. I've also pushed a corresponding branch you can test on Travis.

CommentFileSizeAuthor
default-published.patch746 bytesCrell
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Crell created an issue. See original summary.

timmillwood’s picture

timmillwood
Primary language English
Location 🏴󠁧󠁢󠁷󠁬󠁳󠁿 Wales, UK
CreditAttribution: timmillwood for Pfizer, Inc. commented

I can't see any issues with this change.

Crell’s picture

Crell CreditAttribution: Crell at Palantir.net for Acquia commented
Assigned: Unassigned » timmillwood

Tim: Since the canonical tests are over on Travis, which I don't have access to, I'll let you commit this once it's verified to not break anything inadvertently.

  • timmillwood committed c10ad20 on 8.x-1.x authored by Crell 
    Issue #2704925 by Crell: Trigger workbench merges on published state,...
timmillwood’s picture

timmillwood
Primary language English
Location 🏴󠁧󠁢󠁷󠁬󠁳󠁿 Wales, UK
CreditAttribution: timmillwood for Pfizer, Inc. commented
Status: Needs review » Fixed

Doesn't seem to cause any issues, although we don't have test coverage for the WBM event.

Status: Fixed » Closed (fixed)

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

Status: Closed (fixed) » Needs work

The last submitted patch, default-published.patch, failed testing.

The last submitted patch, default-published.patch, failed testing.

The last submitted patch, default-published.patch, failed testing.

Crell’s picture

Crell CreditAttribution: Crell at Platform.sh commented
Status: Needs work » Closed (fixed)