search

nafiesl / SimpleCrudGenerator

Faster test-driven development for CRUD feature in Laravel project.
https://packagist.org/packages/luthfi/simple-crud-generator
MIT License
170 stars 51 forks source link

Generating CRUD wrappers around existing models #24

Closed pavel-lexyr closed 3 years ago

pavel-lexyr commented 4 years ago

As described. A possible usecase for the library is wrapping an already built model into a CRUD system - resolving this issue will imply that the library can do so.

nafiesl commented 4 years ago

Hi @Ludar-Pavel, ok got your point. You say we need to handle this. Instead of returning an error/warning, we want to generate the CRUD feature anyway (or asking confirmation to the user first).

screenshot_2020-11-03_001

Correct?

pavel-lexyr commented 4 years ago

Yep. That's more of a feature request than a bug, though - the current behaviour seems to be intentional.

nafiesl commented 4 years ago

the current behaviour seems to be intentional.

Yes correct @Ludar-Pavel.

nafiesl commented 4 years ago

hello @Ludar-Pavel could you please try again with dev-master version? 

$ composer require luthfi/simple-crud-generator dev-master --dev

Then try to generate the CRUD files from an existing model class.

screenshot_2020-11-14_012

nafiesl commented 3 years ago

Resolved on https://github.com/nafiesl/SimpleCrudGenerator/releases/tag/1.5.0