Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Switch to using a repository & facade for Redirect, add eloquent support #139

Merged

Conversation

ryanmitchell
Copy link
Contributor

This PR adds eloquent support to the redirects.

To do this I've:

  • Moved any calls to Data\Redirect to a facade (Facades\Redirect)
  • Created a repository contract and query builder contract
  • Created a stache repository and moved the save/delete/query/find methods from Data\Redirect to it
  • Created eloquent repositories

Let me know if you need/want anything else to have this approved.

Closes #106

@ryanmitchell ryanmitchell changed the title Switch to using a repository & facade for Redirect Switch to using a repository & facade for Redirect, add eloquent support Nov 13, 2023
@riasvdv riasvdv merged commit 25255f8 into riasvdv:main Nov 29, 2023
@riasvdv
Copy link
Owner

riasvdv commented Nov 29, 2023

Amazing, thank you!

@ryanmitchell ryanmitchell deleted the feature/option-of-eloquent-driven-redirects branch November 29, 2023 13:49
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support Eloquent redirects
2 participants