Updated: Comment #0

Problem/Motivation

#2052995: Add a route to the frontpage introduced this test but the workaround it implemented became unnecessary in the meantime due to #2062745: Automatically set the request context on the url generator.

Proposed resolution

Remove the workaround.

#2052995: Add a route to the frontpage
#2062745: Automatically set the request context on the url generator

CommentFileSizeAuthor
#1 2070791-1-frontpage-test-request-context.patch1.06 KBtstoeckler

Comments

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
commented
Status: Active » Needs review
StatusFileSize
new 2070791-1-frontpage-test-request-context.patch1.06 KB
tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
commented
Component: base system » routing system
Crell’s picture

Crell commented
Status: Needs review » Reviewed & tested by the community

I like patches that consist only of minus signs.

dawehner’s picture

dawehner
Primary language German
commented

+1

catch’s picture

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

Committed/pushed to 8.x, thanks!

Status: Fixed » Closed (fixed)

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