Problem/Motivation

Workflow routes say 'Edit state', 'Delete state', etc. even for transitions.

Proposed resolution

Change the routes to 'Edit transition', 'Delete transition', etc.

Remaining tasks

Review.

User interface changes

Route titles will be correct.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#3 2840975-3.patch1.25 KBGábor Hojtsy
#3 interdiff.txt513 bytesGábor Hojtsy
#2 2840975.patch1.21 KBGábor Hojtsy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Gábor Hojtsy created an issue. See original summary.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
FileSize
2840975.patch1.21 KB
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
FileSize
interdiff.txt513 bytes
2840975-3.patch1.25 KB

Huh, fixing the newline at end of file where we are here.

timmillwood’s picture

timmillwood
Primary language English
Location 🏴󠁧󠁢󠁷󠁬󠁳󠁿 Wales, UK
CreditAttribution: timmillwood at Appnovation for Pfizer, Inc. commented
Status: Needs review » Reviewed & tested by the community

Thanks, looks good.

The last submitted patch, 2: 2840975.patch, failed testing.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 3: 2840975-3.patch, failed testing.

timmillwood’s picture

timmillwood
Primary language English
Location 🏴󠁧󠁢󠁷󠁬󠁳󠁿 Wales, UK
CreditAttribution: timmillwood at Appnovation for Pfizer, Inc. commented
Status: Needs work » Reviewed & tested by the community

Unrelated fail.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 3: 2840975-3.patch, failed testing.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy at Acquia commented
Status: Needs work » Reviewed & tested by the community

Still an unrelated fail.

alexpott credited Dries.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Status: Reviewed & tested by the community » Fixed

I was told @dries actaully spotted this bug so crediting him :)

Committed faa4016 and pushed to 8.3.x. Thanks!

  • alexpott committed faa4016 on 8.3.x 
    Issue #2840975 by Gábor Hojtsy, Dries: Quickfix: Workflow routes have 'X...

Status: Fixed » Closed (fixed)

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