-
-
Notifications
You must be signed in to change notification settings - Fork 131
Remove deprecated ReST API with dependencies #780
Conversation
Signed-off-by: Wojciech Błoszyk <wbloszyk@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should try to remove datagrid bundle too.
I think about it too. In NewsBundle we use pagination in frontend. We can remove datagrid bundle in favor of KnpPaginationBundle. I will wait with it for now. |
The DatagridBundle pagination was supposed to be only used for API. For frontend, there is already a Pager/Datagrid class in the AdminBundle (and his persistence bundle), made for this. |
Signed-off-by: Wojciech Błoszyk <wbloszyk@gmail.com>
Ready to review. |
Signed-off-by: Wojciech Błoszyk <wbloszyk@gmail.com>
Thanks |
Subject
Remove ReST API with dependencies and update routing with tests.
Remove DatagridBundle with replace Pager by custom Pager (to allow use news without AdminBundle)
I am targeting this branch, because this should be done in unstable branch.