Clear up use of OrientDB as a secondary connection option

sgpatil / oriquent

Orientdb Eloquent driver for Laravel 5

MIT License

47 stars 15 forks source link

Clear up use of OrientDB as a secondary connection option #38

Closed CristianSitov closed 7 years ago

CristianSitov commented 7 years ago

clear up explanation on how to use OrientDB as a secondary connection option
force use of OrientDB connection in migration stubs & generated migration files

sgpatil commented 7 years ago

Hello @CristianSitov , Thank you for contributing to the project. I am okay with the changes Schema/Builder.php Schema/Grammars/OrientdbGrammar.php

But I don't think it would be needed to provide connection in migration stubs files, as it is an optional command.

CristianSitov commented 7 years ago

Hey @sgpatil, I fully understand you. I reverted the changes.

CristianSitov commented 7 years ago

Quick note: once you consider that these 3 commits can be merged, my future contributions will be done on separate branches. Unfortunately, I just observed that I made the commits on the master branch.