search

franznkemaka / fluidbm-cli

⚡ Fluidbm CLI: Import Laravel schema designs to your project with just one command
MIT License
16 stars 0 forks source link

Model relationship properties fail to save #1

Open batfink5000 opened 2 years ago

batfink5000 commented 2 years ago

Hi

I really like this project, great work.

When trying to create a relationship between 2 models the properties I set are not being saved. I have managed to create a relation ship between model A and model B, however when I create a relationship between model C and model D the properties fail to store.

Thanks

batfink5000 commented 2 years ago

Hi I have resolved this now. I was zoomed out and this seemed to be causing some issues.

Thanks.

franznkemaka commented 2 years ago

Hi @batfink5000,

happy to hear that you like the project 🤩

Great that you managed to resolve the issue, shall it repeat again, don't hesitate to re-open the issue.

What would like to have on Fluidbm?

regards!

batfink5000 commented 2 years ago 
Hi

I think this project could be really useful, and following your question I have given it some thought.
Thanks for the project and your hard work.  Here are some of the features and enhancements I would like to see.

Regards,
BF

More information in the model editor window.  (Tabs or on a scrollable window maybe.)

When a model is selected :
    Display its code in a section in the model editor window.
    Include a list of the relationships it owns in the model editor window.
    Relationships (in the main window) should be highlighted/brought to the foreground.

Option to download as a zip all models and relationships.

The foreign keys field should offer a drop-down containing the columns that could be used.

When starting a project offer an option to start it with the standard Laravel auth schema included.

Include a model gallery, this would provide common models, such as address, customer, products etc.  
They could be added from the gallery to provide a starting point.  Its not the relationships so much rather the table designs.

Consider the Laravel naming convention for tables, classes etc and use this to pre-fill fields.
franznkemaka commented 2 years ago

Hi @batfink5000,

thanks for the compliment, I appreciate them.

That sounds like great features to implement, I'll give them a thought and keep you on the hook

regards!

About the issue, as well, I'll be investigating it.

franznkemaka commented 1 year ago

Hi @batfink5000,

thank you for your suggestions. I've noted them and they will be soon built 👍