Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Updating raw DB queries to include DB prefix. #127

Merged
merged 1 commit into from
Sep 25, 2016
Merged

Updating raw DB queries to include DB prefix. #127

merged 1 commit into from
Sep 25, 2016

Conversation

bretvanhorn
Copy link
Contributor

@bretvanhorn bretvanhorn commented Sep 22, 2016

Noticed my CRUD queries were throwing errors and realized the queries were not taking into consideration the database prefix set in my database.php config file. Added in lines to include that.

Noticed my CRUD queries were throwing errors and realized the queries were not taking the database prefix set in my database.php config file. Added in lines to include that.
@tabacitu tabacitu merged commit 4f37d8a into Laravel-Backpack:master Sep 25, 2016
@tabacitu
Copy link
Member

Hi @bretvanhorn ,

Big thanks for the contribution! I'm not usually using db prefixes on Laravel projects, so I naturally missed that.

Cheers, keep 'em coming!

@bretvanhorn
Copy link
Contributor Author

Hey @tabacitu, no problem! Thanks so much for making the Backpack platform. I'm using a lot of your components and they've made my project go much more smoothly. Will definitely submit any other fixes I come across.

@tabacitu
Copy link
Member

:-) it makes me very happy to hear that. Cheers!

@bretvanhorn bretvanhorn deleted the patch-1 branch November 2, 2017 19:49
This was referenced Apr 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants