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

Migrate from kohana to laravel / lumen #1372

Closed
rjmackay opened this issue Sep 8, 2016 · 10 comments
Closed

Migrate from kohana to laravel / lumen #1372

rjmackay opened this issue Sep 8, 2016 · 10 comments

Comments

@rjmackay
Copy link
Contributor

rjmackay commented Sep 8, 2016

This will probably be a multi step process

  1. Install laravel, and migrate routing to laravel. All validation, db queries still use Kohana. Use both auto loaders side-by-side. The big lift here is just migrating the Rest controller.
  2. Start migrating away from any helpers etc we're using. Try to avoid auto loading anything from kohana. DB and validation code could still use kohana.. but we should try to rely on just those libraries and nothing else
  3. Migrate validators
  4. Finally migrate DB code.
@rjmackay
Copy link
Contributor Author

rjmackay commented Feb 7, 2017

@jshorland This got slightly more urgent now Kohana is taking everything offline as of July http://discourse.kohanaframework.org/t/kohana-retirement-2017-07-01/1277

@rjmackay
Copy link
Contributor Author

rjmackay commented May 3, 2017

Progress

  • Add lumen in to codebase Migrate to Lumen #1663
  • Migrate API controllers to Lumen Migrate API controllers to lumen #1674
  • Move Repos, Validators, etc out of Kohana Move code out of kohana application #1741
  • Move DI set up out of Kohana (into a service provider)
  • Remove Kohana dependencies from everything other than validators and repos
  • Stop bootstrapping kohana
  • Fix Dataprovider callbacks
  • Migrate event handling/listeners to lumen
  • Migrate console to artisan
  • Make artisan migrate call phinx
  • Migrate validators to use lumen
    • Add test coverage
    • Use Illuminate\Validation
  • Migrate repos
    • Do we refactor too?
    • Or just move to Illuminate\Database

@rjmackay
Copy link
Contributor Author

Looking at the kohana code. I think it might be possible to extract just the Kohana Validation libs into their own library while we transition, and removing the Kohana dependencies from the database lib should be fairly simple too.

@rjmackay
Copy link
Contributor Author

rjmackay commented Jun 1, 2017

@willdoran here's where this is at:

If you start on anything, just branch off the lumen branch and prefix your branch with lumen-.

@gausam
Copy link
Contributor

gausam commented Jan 5, 2018

@rjmackay Hey I hadn't seen this one. I ❤️ it!

Let me know if I can help in any way. Laravel is a daily eatz for me in several production environments. I'm sure I can make room of at least 4 or 5 hours for you each week.

@caharding
Copy link
Contributor

caharding commented Apr 9, 2018

@rjmackay can you break this out into what you'd like to accomplish this cycle?

@caharding caharding changed the title EPIC: Migrate from kohana to laravel / lumen Migrate from kohana to laravel / lumen Apr 19, 2018
@rjmackay
Copy link
Contributor Author

rjmackay commented Apr 23, 2018

Outstanding questions (leaving this for myself):

  • Should I change httpdocs to public to match the standard laravel convention?
  • Can I replace artisan migrate with phinx?

@rjmackay
Copy link
Contributor Author

rjmackay commented May 2, 2018

@tuxpiper to make migrations easy I think I'll keep the using httpdocs directory. However to follow the lumen conventions I might symlink public to httpdocs. Does that sound reasonable?

@rjmackay
Copy link
Contributor Author

rjmackay commented May 2, 2018

This is now primarily just waiting for QA

@tuxpiper
Copy link
Member

tuxpiper commented May 2, 2018

@rjmackay that sounds totally reasonable to me. I've approved in the PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants