search

InfyOmLabs / generator-builder

InfyOm Laravel Generator GUI Builder
http://labs.infyom.com/laravelgenerator/
MIT License
133 stars 71 forks source link

Generate CRUD from database option GUI ... Update builder.blade.php #44

Open Aramics opened 4 years ago

Aramics commented 4 years ago

Generate from Database Option GUI

ajayinfyom commented 4 years ago

Hi @Aramics nice work.

but there need to some refactor. 1) Select only one table at time from drop down instant of multiple tables and generate scaffold for single table. 2) Model Name option should be there and its required. everyone not want same Model name as table name. 3) Remove commented code like var_dump

btw, You can Put all things in single PR with all changes like route, builder.blade & controller, no need to put separate PRs.

Aramics commented 4 years ago

Thanks for being welcoming @ajayinfyom . Why do you prefer single table scaffolding ? Is it because of the model name ? if Yes , table UI could be with respective model name i.e select box with alternative text box infront for model name if table name is not desired.

if No, then please make clear why since you know every bit of the code . Multiple table scaffolding could be every helpful.

Thanks.

ajayinfyom commented 4 years ago

@Aramics Its fine to generate scaffold with multiple tables at a time.

but , i see here generated model name smaller & plural case. Modal name Must be Singular and Capitalised. Can you please make this changes ?

ajayinfyom commented 4 years ago

@Aramics Its fine to generate scaffold with multiple tables at a time.

but , i see here generated model name smaller & plural case. Modal name Must be Singular and Capitalised. Can you please make this changes ?

i fixed that things in second PR : https://github.com/InfyOmLabs/generator-builder/pull/47/commits/10f2a97c7653ce4f63fca3fd433676c8dc9a00c3