Skip to content
This repository has been archived by the owner on Apr 20, 2021. It is now read-only.

Decoupling improvements #5

Open
wants to merge 22 commits into
base: master
Choose a base branch
from

Conversation

tiagoalves
Copy link
Contributor

The Laravel 5.1 compatibility branch (lvl51) has introduced several decoupling improvements which facilitate the usage of this module by third-party Laravel applications.

These include:

  • The base application can provide it's own User as long as it includes this module's provided UserTrait.
  • The base application can provide its own API to BLM dispatcher.
  • A module-specific exception handler so that we can provide prettier error messages.

This PR migrates these improvements to the master branch where we maintain the Laravel 5.2+ compatibility.

Koen Betsens and others added 22 commits December 2, 2016 07:54
The Laravel 5.1 Branch is not intended to merge, but rather to reserve
a lane for those 5.1 projects that are not ready yet to move to 5.2
Does not exist in 5.1
Devs can now provide their own job dispatcher implementation.
Devs can now provide their own user model implementation.
Added indexes to the `oauth_access_tokens` table.
Now, base apps can provide their own logic to create a user
representation object with the fields that this module's views require.
Now we can return error messages with the usual UI styling of this
package.
…uirements.

* The User model and api->blm job dispatcher can now be provided by the
base Laravel app.
* Package-specific exception handler so we can transform exceptions into
errors that are shown in the login view instead of dumping a stack trace
in the browser.
*
* Added indexes to the `oauth_access_tokens` table for faster lookups.
Now the module uses its own User trait as well.
Change introduced in 99af026.

This branch is for Laravel 5.2+, that's why I'm removing the Laravel 5.1
related change that was merged into this branch.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant