JQuery gets 404 in configurator about actions-for-nautilus

[maygeisler]

or you just leave out the line where you serve jquery entirely bc you're serving everything in the ./javascript/ folder anyway in the line below. Just strange that it doesn't work for me at all when the path is there and wrong.

[bassmanitram]

Hey... Sorry I haven't got back on this yet. I'll take a look this week, I hope

…

On Fri, 17 Nov 2023, 23:42 Maileen Geisler, ***@***.***> wrote: or you just leave out the line where you serve jquery entirely bc you're serving everything in the ./javascript/ folder anyway in the line below. Just strange that it doesn't work for me at all when the path is there and wrong. — Reply to this email directly, view it on GitHub <#49 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADAKLWUCTKSIWPB7EOHNQJTYE7R4HAVCNFSM6AAAAAA7IA3IOWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBXGEZTOOBZHA> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[bassmanitram]

OK, @maygeisler, I've taken a look :) (finally)

JQuery is a prerequisite when installing the extension. This gets enforced when installing by the DEB file, in which case it gets put into /usr/share/javascript/jquery/jquery.min.js.

The reason I do that rather than just redistribute myself is because when constructing the DEB file I get a nasty warning that I shouldn't do that when the package has its own DEB file.

This means that if you install the DEB file, you should get the jquery stuff properly installed too. But if you install via the install script ... theeeennnn it's a bit messier. It is listed as a prereq in the readme, but on non-DEB systems, if it has its own package it still may get placed somewhere else.

However, I think I can mess with the installation process to have jquery there, NOT include it in the DEB and sort out the non-DEB installation to work properly.

I'll get onto that.

Thanks for using the extension.

[bassmanitram]

@maygeisler - I fixed it in the "Main" branch. Since it only affects "make" installs, I won't make a new release solely for this issue but will include it in the next release that includes other issue resolutions and features.