FaisalUmair / udemy-downloader-gui

A desktop application for downloading Udemy Courses
MIT License
6.14k stars 1.72k forks source link

Translation, download path selection and stop/resume #2

Closed m4dbra1n closed 6 years ago

m4dbra1n commented 6 years ago

Hi,

I'm an Italian user and I've discovered this little piece of bits just today :D

As from the title I have 3 requests:

Thanks and keep up the good work! :D

FaisalUmair commented 6 years ago

Hi, welcome to this project and thank you for taking the time to help improve this software.

As for the progress, I am currently working on the Stop/Pause/Resume functions. I recently upgraded to a new download manager package which allows to pause and resume downloads. However, I have not implemented the pause and resume functionality from the package yet as I wanted to give it a little more time with the basic download to see if it works fine.

I also have some settings to add which include the download path selection.

Now coming to the Italian translation part, I will surely take your help with that once I finish implementing the pause/resume functionality.

Once again, thank you for your contribution 👍

FaisalUmair commented 6 years ago

The Download location option has been added in https://github.com/FaisalUmair/udemy-downloader-gui/commit/ae7b74aeb28d7fce97d405c2a79b12931e2afa3b , will be available in next release. Download Location Screenshot

FaisalUmair commented 6 years ago

I have added the download stop/resume and translation option.

pause-resume language

Now for the translation part, if you would like to contribute to Italian translation. Here are the steps to follow:

1. Fork this repository.
2. Update the file locale/it.json and add Italian translation to the given English words.
3. Make a pull request.

I am planning to release v1.2.0 with translation support (at least Italian for now), I can use google translate tool for this but human translation would be way better.

Thanks 👍

m4dbra1n commented 6 years ago

OK, I've forked and then made a pull request, but seems to be on my fork of the App, not your main. Can you kindly check if I did it well, in case? Sorry for my noobness on GitHub :(

FaisalUmair commented 6 years ago

Thanks for the quick response.

I think you have made a pull request to your fork. You do not need to go to the pull request tab. On the main page of your fork, you can see a pull request option (where the latest commit is displayed). You have to click that option and not the pull request tab on your fork.

m4dbra1n commented 6 years ago

OK, I've tried to do a pull request with the file correctly edited directly from your repository, without cloning it. PS: maybe we could communicate better on Telegram, if you like :)

FaisalUmair commented 6 years ago

Yeah that works as well. Thanks 👍

I don't actually use Telegram. But you can contact me on my email address. I have it listed on my profile.

For chat, Google Hangouts will do the job.

FaisalUmair commented 6 years ago

The translation works fine. Thanks for contributing.

So the download path, stop/resume and translation all are implemented. Shall I close this issue now?

m4dbra1n commented 6 years ago

All implemented, why not then? ;)