nuxt-open-fetch icon indicating copy to clipboard operation
nuxt-open-fetch copied to clipboard

fix: reactive query params

Open adamdehaven opened this issue 1 year ago • 3 comments

Modify the ComputedOptions interface to allow for reactive query params as explained by #52 .

@enkot this should be tested to ensure I didn't miss something with the new interface (and that it doesn't impact other parameters properties).

adamdehaven avatar Jul 01 '24 00:07 adamdehaven

@adamdehaven is attempting to deploy a commit to the Taras Batenkov's projects Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Jul 01 '24 00:07 vercel[bot]

@enkot is there anything I can do to help move this PR along, or adjust as necessary?

adamdehaven avatar Jul 08 '24 16:07 adamdehaven

@enkot hoping to bump this PR one last time.

Please let me know if I can add more detail or if I missed something in the implementation.

adamdehaven avatar Sep 24 '24 01:09 adamdehaven

Is there a way I can patch my npm installed version with this change?

haringsrob avatar Oct 24 '24 18:10 haringsrob

Is there a way I can patch my npm installed version with this change?

You could override the module types if you really wanted to

adamdehaven avatar Oct 25 '24 00:10 adamdehaven

Dang really hoped this would make it to the latest release.

@enkot is there a reason not to have this? Or can you provide docs on how to have reactive props?

haringsrob avatar Nov 01 '24 07:11 haringsrob

Hi @haringsrob, Should be fixed in the latest release👌

@adamdehaven Thank you for your PR and help with other issues! 💪

enkot avatar Nov 01 '24 08:11 enkot

Thank you indeed it works. Did not notice it from the changelog.

haringsrob avatar Nov 01 '24 08:11 haringsrob