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

Crud Issue #49

Closed
adilsaeed31 opened this issue Jan 25, 2016 · 4 comments
Closed

Crud Issue #49

adilsaeed31 opened this issue Jan 25, 2016 · 4 comments

Comments

@adilsaeed31
Copy link

HI,

I need some information, your crud generator is really awesome, well let me know i don't to generate the migration, i need only controller, model and views only, also route. but when i generate crud through your command, then in model my table name is automatically called with factories, instead of factory. so that is issue number 1.

2nd most important thing is when you write only (php artisan crud:generate TableName) and press enter it only create model/controller and route. why not create view, it should create view based on table columns, if table columns is already exist. so this is my suggestion please add this feature.

3rd thing after giving complete columns through command then your view only create four table heading in listing view plus migration which i don't need if i have already migration class exist.

4th thing what about foreign keys if we want dropdown as select tag then in your view you need to pull data from database table and display as option in select tag,
for example:
I have country field like pakistan, bangladesh, and their values are 1, 2 so i need them while create.

Hope you understand.

Please let me know.

Best Regards
Muhammad Adil

@sohelamin
Copy link
Owner

Thanks @adilsaeed31 for your suggestion and nice explanation.
I will do some changes near future but everything will be not possible to make cause I've to do others works.
Any way, PR are always acceptable.

Thanks again :)

@adilsaeed31
Copy link
Author

Thanks for your reply, can i contribute to complete this.

On Fri, Jan 29, 2016 at 1:08 PM, Sohel Amin notifications@github.com
wrote:

Thanks @adilsaeed31 https://github.com/adilsaeed31 for your suggestion
and nice explanation.
I will do some changes near future but everything will be not possible to
make cause I've to do others works.
Any way, PR are always acceptable.

Thanks again :)


Reply to this email directly or view it on GitHub
#49 (comment)
.

@sohelamin
Copy link
Owner

Sure.

@sohelamin
Copy link
Owner

@adilsaeed31 you can check now I've already fixed some issues.

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

No branches or pull requests

2 participants