rss-bridge icon indicating copy to clipboard operation
rss-bridge copied to clipboard

[Core] add dynamic list with ajax requests

Open floviolleau opened this issue 1 year ago • 8 comments

Hi!

The goal of this PR here is to be able to have dynamic lists for parameters. It introduces a new type for parameters: dynamic_list.

This feature is to answer to some bridges that have big select / options in parameters.

I'm using datalist tag described here: https://www.w3schools.com/tags/tag_datalist.asp).

An example of usage can be seen here: #4362 and #4363

Kind regards

floviolleau avatar Dec 08 '24 22:12 floviolleau

yes i can see how this is useful.

but that javascript inside html tag is not good. can you move it out to the .js file we already have?

the rest im unsure of. will take a closer look

dvikan avatar Dec 12 '24 16:12 dvikan

Hi,

I wasn't sure about how to add a js file inside the project. If you can guide how to do that?

Thanks!

floviolleau avatar Dec 12 '24 17:12 floviolleau

i'd prefer you add a function to the already existing file static/rss-bridge.js

but, using that thirdparty CORS proxy is not good. also did you test if the config proxy works? last time i tried, i could not get proxy to work.

ideally the ajax should go towards rssbridge. not relying on thirdparty.

dvikan avatar Dec 13 '24 03:12 dvikan

Hi,

Yes I will do that for the function and retest.

I agree with you, 3rd party proxy is not good at all. Yes I tested it and proxy config works!

Do you know an endpoint that can do the same in RSS-Bridge or we have to create a new one?

floviolleau avatar Dec 13 '24 09:12 floviolleau

to do this properly i think we should add a new function to BridgeAbstract: getDynamicData(), then the javascript could do ajax towards a new route/action and then fill in the list for autocomplete.

i think it be better if I implement this. but still feel free to try yourself and create PR. i dont have time now but in the next few weeks i can give it a go

dvikan avatar Dec 13 '24 17:12 dvikan

Hi,

Sounds like a plan 😉!

Sure Xmax is coming and we will be all busy :christmas_tree: :santa: .

I will wait your new endpoint and update this PR accordingly once done.

Thanks

floviolleau avatar Dec 16 '24 10:12 floviolleau

Hi,

Any news for an approval?

floviolleau avatar Aug 18 '25 21:08 floviolleau