search

chiiya / filament-access-control

Admin user, role and permission management for Laravel Filament
MIT License
192 stars 24 forks source link

Compatibility with filament-logger #16

Closed cweagans closed 1 year ago

cweagans commented 2 years ago

In order for the activity log exposed by filament-logger to show users managed by this package, the FilamentUser model needs a method like so:

    /**
     * Return a name.
     *
     * Needed for compatibility with filament-logger.
     */
    public function getNameAttribute(): string
    {
        return $this->getFilamentName();
    }

It's a really minor thing, but it ensures that the user who caused an event is clearly visible.

chiiya commented 1 year ago

Happy to approve a pull request if you want to add this!

chiiya commented 1 year ago

Fixed in 1.6.0