- Issue created by @scott_euser
- 🇬🇧United Kingdom scott_euser
This is going to fail as it needs these:
But then still needs work once those are resolved to add a few more autocomplete attempts and prove the restricted results.
- 🇬🇧United Kingdom dahousecat
This is next on the to-do list but I'll need to complete this task in my own time so will have to wait for now.
- 🇬🇧United Kingdom scott_euser
Just to note it's mostly there in the MR. I would be happy to complete it
- 🇬🇧United Kingdom dahousecat
Thanks again Scott, I totally missed that.
If your happy to complete that then i would be very grateful.
I'm a bit new to MRs on drupal.org - if you could give me any pointers on how to merge then that would be super helpful!
- 🇬🇧United Kingdom scott_euser
Sure absolutely, I posted here in one of the issues you merged with details: https://www.drupal.org/project/entity_reference_field_autocomplete_filte... →
- 🇬🇧United Kingdom scott_euser
Tests now passing. Note that in the MR is a slight change to #3525063: Widget missing schema → as the schema was still throwing a validation error (test coverage automatically triggers config validation)
-
dahousecat →
committed 184fc5c7 on 1.0.x authored by
scott_euser →
Issue #3525066 by scott_euser: Add automated test coverage
-
dahousecat →
committed 184fc5c7 on 1.0.x authored by
scott_euser →
- 🇬🇧United Kingdom dahousecat
Thanks Scott - runs fine locally.
Really appreciate you adding this test.
Automatically closed - issue fixed for 2 weeks with no activity.