Support base fields as well.

CommentFileSizeAuthor
#3 2721319-3.patch3.17 KBbenjy

Comments

Xano created an issue. See original summary.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented

#2633800: Add a Views filter for plugin fields does the same thing, but for a different field type. We can use the solution there for this issue.

benjy’s picture

benjy commented
Status: Active » Needs review
StatusFileSize
new 2721319-3.patch3.17 KB

I made a start on this if anyone needs it.

Status: Needs review » Needs work

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

bradjones1’s picture

bradjones1
Primary language English
commented

Using this and it still applies fine and works (so far...) as advertised. What's with the CI error on the tests? Does this just need a re-run or do the maintainers want further edits?

benjy’s picture

benjy commented
Status: Needs work » Needs review
dietr_ch’s picture

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

Working as intended.

kgaut’s picture

kgaut commented

working fine thanks !

markdc’s picture

markdc
Location Selters
commented

This has been RTBC for over a year now and no new releases for almost 3 years. Can we at least get this into a next beta release?

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 3: 2721319-3.patch, failed testing. View results

legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
commented
Status: Needs work » Fixed

Committed and pushed to 8.x-1.x. Thanks!

  • legolasbo committed 6d4ae96 on 8.x-1.x authored by benjy 
    Issue #2721319 by benjy, bradjones1, dietr_ch, fatmarker, Kgaut,...

Status: Fixed » Closed (fixed)

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