johman10 / flood-for-transmission

A Flood (https://github.com/Flood-UI/flood) clone for Transmission
GNU General Public License v3.0
339 stars 33 forks source link

Update config.json to support _all_ User Interface options #514

Closed defkev closed 10 months ago

defkev commented 1 year ago

not just common download paths. This expands on @fastiuk's PR (#502) by exporting config.json through a helper which makes it universally accessible in every module as e.g. import config from configHelper. The "default" config.json is equal to the hard-coded settings which are left in place in case a property is missing from config.json COMMON_PATH has been updated to an array since transforming it between a string and an array is redundant (it's not a cookie) README update included to reflect the changes.

Cheers

Edit: This isn't exactly what #434 is about but close enought.

johman10 commented 1 year ago

Wow! Thanks @defkev! I will aim to have a look at this soon!

johman10 commented 10 months ago

Hi @defkev as you can see I created a seperate PR and implemented some small things as discussed here. Basically this commit is what changed and what I tested: https://github.com/johman10/flood-for-transmission/pull/540/commits/9a44c5da3e0fde6fb2f3e3b4edd33c5c33901f51