Secretmapper / combustion

Combustion is a sleek, modern web interface for Transmission
http://secretmapper.github.io/combustion/
Other
461 stars 36 forks source link

Torrent Destination Folder not pre-filled with downlad-dir path #45

Open OscarVanL opened 4 years ago

OscarVanL commented 4 years ago

Transmission has a number of CLI configuration parameters (see here), one of which is download-dir. This sets the 'default' download directory.

Using the default or kettu web UIs this box is prefilled to the value of download-dir. See screenshot: Capture2 (Download directory is pre-filled) Whereas in combustion UI this does not happen: Capture3 (Download directory is empty, displaying NaN TB free)

I think combustion should autofill this box based on the download-dir parameter.

Observing the source code, it appears this is the intended behaviour this.torrentUpload.setDownloadDir(this.props.session_store.settings['download-dir']); in index.js, however for some reason this does not happen.

ozamorowski commented 4 years ago

this.props.session_store.settings is empty on inital render. I used when function of mobx to wait until settings will be populated.

OscarVanL commented 4 years ago

Nice, good job finding the cause :)