search

discoverydesign / filament-gaze

👀 See who's viewing a resource in Filament PHP 🔭
MIT License
119 stars 7 forks source link

Problems with ldap-record #11

Closed sinnbeck closed 3 weeks ago

sinnbeck commented 3 weeks ago

I am working on a project that uses ldap-record for auth. That is presenting some problems with this package sadly.

https://github.com/discoverydesign/filament-gaze/blob/main/src/Forms/Components/GazeBanner.php#L186

Here it uses the LdapModel which means on line 194 it searches on ldap, instead of the database.

A suggestion would be to allow us to set the class to use in either a config file or when instantiating the plugin

Let me know if you would want me to make a PR :)

discoveryjames commented 3 weeks ago

Hey @sinnbeck

I've refactored things a little and it should resolve the issue :)

Give 1.1.5 a go and let me know if you have any issues.

sinnbeck commented 3 weeks ago

Awesome. I'll give it a try as soon as possible!