outl1ne / nova-detached-filters

This Laravel Nova package allows you to detach filters from the filter dropdown and show them on a card
MIT License
61 stars 20 forks source link

Any chance this package might be updated for Nova 4? #24

Closed ziming closed 2 years ago

ziming commented 2 years ago

Any chance this package might be updated for Nova 4?

Thank you! Love your other packages so far

eugenefvdm commented 2 years ago

Also super keen. Wish I had the knowledge to do a pull request, but the VueJS stuff goes over my head. For me the killer feature is the ability to save the state of the search.

mateusz-peczkowski commented 2 years ago

1+

philipbaginski commented 2 years ago

1+ :-)

internetbug256 commented 2 years ago

Same here. I still don't understand why Nova authors aren't considering the fact that we all need and use persistent filters.

ycoloma-etionic commented 2 years ago

+10

KasparRosin commented 2 years ago

We have quite a big backlog of packages that require upgrading to support nova v4 (we had total of 34 nova-* packages, some by now have been updated to support nova v4). Currently we updating them one by one based on the internal usage and sponsored requests.

Pull requests are welcome.

KasparRosin commented 2 years ago

Created v4 branch, where you can track the progress.

Current progress:

philipbaginski commented 2 years ago

Created v4 branch, where you can track the progress.

Current progress:

  • [x] The component can be used (supports vue3, filters are applied)
  • [x] Filter state is being saved to query parameters
  • [ ] Filter state is persisted to localStorage
  • [ ] Filter state is synced with dropdown filter menu
  • [ ] Clear filter and other helpers work as expected and are tested.
  • [ ] Update broken styles
  • [ ] Support dark theme

Is it ready to install with Nova 4 or we have to wait a little bit?

eugenefvdm commented 2 years ago

@philipbaginski it looks like that current progress has 2/7 items completed. So I imagine 5 items are incomplete, and it would be better to wait or do a pull request and see if you can work on any of them.

philipbaginski commented 2 years ago

@philipbaginski it looks like that current progress has 2/7 items completed. So I imagine 5 items are incomplete, and it would be better to wait or do a pull request and see if you can work on any of them.

I would like to help, but I'm newbie and I'm useless for you :-)

KasparRosin commented 2 years ago

Added another patch of changes. Currently there are two things that in-progress/not-ready.

  1. Resources per page selector.
  2. Filter syncing.

I'll do a final round of testing today/tomorrow and then create a release and update documentation. It should be usable now.

KasparRosin commented 2 years ago

Regarding filter syncing, there seems to be an issue where if you change or reset filters from the dropdown menu, the filters in detached card will show old values and not update. Same will apply vise-versa. Because filters are built on vuex, I'm assuming there might be an issue within the store implementation somewhere.

I'm pretty sure this worked in version 3 of laravel nova, so I'll need to debug some more, to see where is the problem.

ycoloma-etionic commented 2 years ago

thanks for the quick update man, cant wait to try it out

philipbaginski commented 2 years ago

Added another patch of changes.

Currently there are two things that in-progress/not-ready.

  1. Resources per page selector.

  2. Filter syncing.

I'll do a final round of testing today/tomorrow and then create a release and update documentation.

It should be usable now.

😍

KasparRosin commented 2 years ago

Created an issue in the nova-issues repository regarding filters and their values. Hopefully somebody from there can shed some additional light as to why it broke with nova 4. laravel/nova-issues#4495

ziming commented 2 years ago

I see you have merged and tagged it, does it mean it is fixed already? :D

KasparRosin commented 2 years ago

The filter syncing is not fixed, but due to it not affecting most users, I decided to release 2.0.0 anyways.

KasparRosin commented 2 years ago

Check out the release notes. Nova 4 support available with release 2.0.0.

Quite a few things changed, most affecting is the namespace change that comes with the release. https://github.com/outl1ne/nova-detached-filters/releases/tag/2.0.0

I'll close this and open two new separate issues for the remaining problems.

KasparRosin commented 2 years ago

Moving any further discussion about nova 4 support to this discussion. https://github.com/outl1ne/nova-detached-filters/discussions/27

philipbaginski commented 1 year ago

But: I have to date filters and reset works with them :-) Bright side of the issue ;-) Any progress with Nova devs to fix this problem?