Use user identifier accessors if applicable

spatie / activitylog

A very simple activity logger to monitor the users of your website or application

https://spatie.be/opensource/laravel

MIT License

582 stars 75 forks source link

Use user identifier accessors if applicable #47

Closed Propaganistas closed 8 years ago

Propaganistas commented 8 years ago

This is more robust (tests are not updated).

Also, you should really update the dependencies in composer.json, as the package is using components of illuminate/config and illuminate/database. Moreover, the package assumes that Laravel's shipped default aliases are still present (e.g. Eloquent, Config, Log). IMHO packages are better off to use the Facade directly.

freekmurze commented 8 years ago

Hi, the tests seem to fail for this PR.

I'm currently working on the next major release of this package. When it comes out (in the coming weeks) this package will be marked as abandoned.

freekmurze commented 8 years ago

Hi, this package has been abandoned. Please use laravel-activitylog instead.