Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

A single configuration file for all settings #23

Open
probstlukas opened this issue Oct 28, 2021 · 3 comments
Open

A single configuration file for all settings #23

probstlukas opened this issue Oct 28, 2021 · 3 comments
Labels
enhancement New feature or request

Comments

@probstlukas
Copy link
Contributor

Files such as .iliasignore, .iliaslogin, course_names.toml and the flags/options that can be set via the command itself should be combined in a single configuration file such as downloader.conf.

@FliegendeWurst FliegendeWurst added the enhancement New feature or request label Oct 30, 2021
@floriensk
Copy link

I would add that the .iliaslogin is a bit unhandy as it has to be present in the target folder, which might be different for each course download. It would be, I think, wiser to search for it (or any config file) in the directory where the executable itself is.

@FliegendeWurst
Copy link
Owner

Good point. However, the .iliasignore configuration probably needs to be different per download target.. maybe it would be best to have a central config file (combining login + course names) and separate configs for each download directory (.iliasignore).

@floriensk
Copy link

That's a good point, I'd love to see this in an upcoming update. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants