Problem/Motivation

Soft blocker for testing for #2409209: Replace all _url() calls beside the one in _l().

Proposed resolution

Display plugins should have an interface.

Remaining tasks

User interface changes

None.

API changes

Addition of an interface for display plugins.

CommentFileSizeAuthor
#2 interdiff-oops.txt536 bytesxjm
#2 vdc-2417865-2.patch37.69 KBxjm
vdc-display-plugin-interface.patch37.75 KBxjm

Comments

Status: Needs review » Needs work

The last submitted patch, vdc-display-plugin-interface.patch, failed testing.

xjm’s picture

xjm
she/her
Primary language English
commented
Status: Needs work » Needs review
StatusFileSize
new vdc-2417865-2.patch37.69 KB
new interdiff-oops.txt536 bytes

Accidentally included an out-of-scope change.

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

Let's get it in

dawehner’s picture

dawehner
Primary language German
commented
Priority: Normal » Major
xjm’s picture

xjm
she/her
Primary language English
commented
Issue tags: +blocker
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Now with new and improved Actually Pushed™ :P

Committed and pushed to 8.0.x. Thanks!

  • webchick committed bd0e53f on 8.0.x 
    Issue #2417865 by xjm: Add a Views DisplayPluginInterface

Status: Fixed » Closed (fixed)

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