philiplb / CRUDlex

CRUDlex is an easy to use CRUD generator for Symfony 4 and Silex 2 which is great for auto generated admin pages
https://philiplb.de/crudlex/
MIT License
109 stars 23 forks source link

Group entities in the navigation Bar #85

Closed dmaciel closed 7 years ago

dmaciel commented 7 years ago

Hi

The idea is to allow to group multiple entities into one option of the menu, this is helpful when there are too many entities or want to group based on relation. There is a example of the result:

navbargroup example
coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling de8a7ddec6486870df073d7fdef92a7a32be8855 on dmaciel:master into 97569c2f18f4873d0adbe988b502b918e6f66400 on philiplb:master.

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling 0346604cb7c91aa76c62b4d2594134a3047f1b32 on dmaciel:master into 97569c2f18f4873d0adbe988b502b918e6f66400 on philiplb:master.

philiplb commented 7 years ago

Hi, thank you for the adjustments. :) One of the tests is failing though: https://travis-ci.org/philiplb/CRUDlex/jobs/223153160

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling 4b84618f52d7b43c59ae102605c41988d220281a on dmaciel:master into 97569c2f18f4873d0adbe988b502b918e6f66400 on philiplb:master.

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-0.08%) to 99.917% when pulling 948d594b95cf60755e72bc2044fb1600c8e4fc52 on dmaciel:master into 97569c2f18f4873d0adbe988b502b918e6f66400 on philiplb:master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.0009%) to 99.917% when pulling 970a08b7c20a38905851dc88522e83a7d9f40699 on dmaciel:master into 19073137c83c20c01eab4ba6c5bfea4e39613b11 on philiplb:master.

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-75.2%) to 24.744% when pulling 970a08b7c20a38905851dc88522e83a7d9f40699 on dmaciel:master into 19073137c83c20c01eab4ba6c5bfea4e39613b11 on philiplb:master.

coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.0009%) to 99.917% when pulling 0b1f947113d8700b7dd86e4a1493b6b5463d70f4 on dmaciel:master into 19073137c83c20c01eab4ba6c5bfea4e39613b11 on philiplb:master.

dmaciel commented 7 years ago

Hi

I already made the requested changes, but looks like this pull request is staled. Can you confirm if everything is ok?

philiplb commented 7 years ago

All good, merged now. :) Thank you very much for this nice feature! :)