nova-detached-actions icon indicating copy to clipboard operation
nova-detached-actions copied to clipboard

Laravel Nova 4.0 Support

Open h888t opened this issue 2 years ago • 19 comments

  Problem 1
    - gobrightspot/nova-detached-actions is locked to version 1.1.1 and an update of this package was not requested.
    - gobrightspot/nova-detached-actions 1.1.1 requires laravel/nova ^3.0 -> found laravel/nova[dev-master, v3.0.0, ..., 3.x-dev (alias of dev-master)] but it conflicts with your root composer.json require (~4.0).
   

when updating to Laravel Nova v4.

h888t avatar Apr 05 '22 07:04 h888t

i dont think its gonna be updated as nova 4 has already this feature implemented

link for documentation

ibrahem-kamal avatar May 03 '22 19:05 ibrahem-kamal

@ibrahem-kamal trouble is that Nova 4 still shows standalone action within the action dropdown, and the good thing this package adds is an ability to make separate action button next to Create Resource button. Less clicks for heavily used actions, better UI.

centgart avatar May 18 '22 13:05 centgart

Bump. Would like to have this package work for Nova 4 as well.

RenoLooijmans avatar Jul 10 '22 15:07 RenoLooijmans

This fork has Nova 4 support

https://github.com/lucidlogic/nova-detached-actions

@lucidlogic Any chance you could take all of your awesome changes in your fork and submit a PR?

vesper8 avatar Jul 13 '22 16:07 vesper8

This fork has Nova 4 support

https://github.com/lucidlogic/nova-detached-actions

@lucidlogic Any chance you could take all of your awesome changes in your fork and submit a PR?

Are you sure his fork works for Nova 4? I have tried it but I couldn't get it to work...

RenoLooijmans avatar Jul 15 '22 15:07 RenoLooijmans

I have also tried to install the fork but couldn't get it to work. Is there anything I may be missing?

georgep-am avatar Aug 02 '22 15:08 georgep-am

yea, the "composer require gobrightspot/nova-detached-actions" doesnt work at all 😅

Hemnyos avatar Aug 03 '22 08:08 Hemnyos

I have created PR for Nova 4 Update. Tested all the features. Working as expected. https://github.com/gobrightspot/nova-detached-actions/pull/74

ali-raza-saleem avatar Aug 18 '22 18:08 ali-raza-saleem

Please review PR above.

matthewhutchings avatar Aug 25 '22 13:08 matthewhutchings

I have created PR for Nova 4 Update. Tested all the features. Working as expected. #74

Hi @the-alichemist. I saw that your PR was merged. Can you please create a tag for this new version? Thanks a lot.

adrianmihaila avatar Sep 05 '22 07:09 adrianmihaila

Also, I found an issue. If I disable the standalone property from a detached action, and I show that action onlyOnIndexToolbar when you select one or two resources for example from the listing and click on the detached action button, on:

public function handle(ActionFields $fields, Collection $resources) {}

you will get all resources. Like you pressed on the select all checkbox. Is there anyone that noticed this issue?

adrianmihaila avatar Sep 07 '22 09:09 adrianmihaila

I created a PR for the issue: https://github.com/gobrightspot/nova-detached-actions/pull/75

adrianmihaila avatar Sep 07 '22 13:09 adrianmihaila

Is there any way to put the button next to the others? Screen Shot 2022-11-13 at 4 21 04 PM

jjjrmy avatar Nov 13 '22 21:11 jjjrmy

Since #75 has been merged, what are we waiting for to create a new release ?

mrleblanc101 avatar Dec 21 '22 21:12 mrleblanc101

#75 doesn't look merged to me...

vesper8 avatar Dec 22 '22 00:12 vesper8

@vesper8 I meant #74, which include Nova 4 support

mrleblanc101 avatar Dec 22 '22 01:12 mrleblanc101

Ah right.. well I agree with you, a new tag would be nice. I've just been using "gobrightspot/nova-detached-actions": "dev-master", for a while now and it does work great.

vesper8 avatar Dec 22 '22 12:12 vesper8

@jjjrmy i got the same issue...rest seems to work just fine

d00p avatar Jan 12 '23 14:01 d00p

Is there any way to put the button next to the others? Screen Shot 2022-11-13 at 4 21 04 PM

@jjjrmy i got the same issue...rest seems to work just fine

same here, did you guys figured out?

wajeht avatar Mar 28 '23 20:03 wajeht