generator-builder icon indicating copy to clipboard operation
generator-builder copied to clipboard

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

Open Aramics opened this issue 6 years ago • 4 comments

Generate from Database Option GUI

Aramics avatar Oct 14 '19 13:10 Aramics

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.

ajayinfyom avatar Oct 19 '19 06:10 ajayinfyom

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.

Aramics avatar Oct 19 '19 15:10 Aramics

@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 avatar Nov 22 '19 11:11 ajayinfyom

@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

ajayinfyom avatar Dec 07 '19 11:12 ajayinfyom