First of all, thanks for all the hard work, New Zeal!
I'm opening a series of feature requests to track progress on this module.

The current widget assumes that users know the name of the View. In certain set-ups with non-technical users or restricted permissions, this can be hard to use, since users would have to "guess" the name of the View they're trying to reference. An additional widget which presents the options as a Select list would solve this issue.

CommentFileSizeAuthor
#4 offer_view_choice_as-2816567-4.patch6.76 KBjoekers
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Algeron created an issue. See original summary.

NewZeal’s picture

Suggestion noted. There are a number of improvements that can be done to the selection process. Thanks.

joekers’s picture

Assigned: Unassigned » joekers

I could do with this on a project I'm working on so I'm going to give it a try.

joekers’s picture

Status: Active » Needs review
FileSize
6.76 KB

I haven't had any chance to take a look at this until today.

Working patch attached but there's one thing I'm not sure of: validateElement removing this and the line that adds it (element_validate) gives this error:
InvalidArgumentException: Value is not a valid entity. in Drupal\Core\Entity\Plugin\DataType\EntityReference->setValue()

Any feedback would be much appreciated :)

NewZeal’s picture

I've pushed this with the alpha 6 release. Because this is still alpha and the plugin is a separate script, then it works to use this method for testing by the community. I also invite you to be a maintainer of the module. Let me know if you want to accept.

joekers’s picture

Yeah I'd like to be a maintainer of the module, thanks!

NewZeal’s picture

Ok, you have full privileges. Thanks. We'll work towards a beta version once we have dealt with this issue: https://www.drupal.org/node/2816571 .

joekers’s picture

Thanks. Sounds good to me.

joekers’s picture

Status: Needs review » Fixed

Closing this issue as the code has been committed and any issues can be reported separately (as with #2832518: "None" Option Throws Error When Trying To Remove Selected View).

Status: Fixed » Closed (fixed)

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