auto-ytdl icon indicating copy to clipboard operation
auto-ytdl copied to clipboard

support channel directory

Open onokatio opened this issue 5 years ago • 1 comments

This PR supports custom directory path for saving music.

  • [x] support custom directory
  • [ ] use chennel name as custom directory

onokatio avatar Aug 15 '20 19:08 onokatio

Welcomed feature as well :-)

However, i think it should be an optionnal feature, and thus hits the same issue as #18 because it would require a config entry.

I think it would have an issue as aytdl can take non-channel urls as input, as well as youtube channels urls that do not contain the channel's name. You should iron out your conditions and edit the config to be reviewed

I'd like to focus on windows support first, but your contributions are welcome. Can you rebase (apply your changes) on the dev branch? it contains most of the cross-platform code so you can imitate it. dev also contains guidelines for testing aytdl works as intended (to make sure your feature implementation do no break the existing features)

michaelb avatar Aug 15 '20 20:08 michaelb