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

L11 compability without doctrine #1101

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

piovezanfernando
Copy link

These changes ensure that the package works with version 11 of Laravel without doctrine support, doctrine would provide us with a much more friendly interface to use the database mapping and its tables, but as this package was removed I made this small change ( which didn't look very pretty but is functional), I intend to adjust the methods so that get/set methods can be used instead of direct queries to the arrays, but for now this is what I could do so that my new project can move forward.

If you want to use the fork until the people responsible for the project approve this PR, here's how to use it

{
     "repositories": [
         {
             "type": "vcs",
             "url": "[email protected]:fernandopiovezan1/laravel-generator.git"
         }
     ]
}
{
     "require": {
         "infyomlabs/laravel-generator": "dev-l11-compability-without-doctrine",
     }
}

Thanks to @laravel-shift for the adjustments to the dependencies, as I have already updated this version. #1098

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.

1 participant