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

Added support to download images from multiple regions. #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

freaksterrao
Copy link

@freaksterrao freaksterrao commented May 26, 2016

  1. If downloading from multiple regions, the code now does not force the wallpaper. Instead it is expected that user sets the wallpaper to be a slide show pointing to Bing Wallpaper\Year folder
  2. Added support for hiding the console window. I use it when I run it as a schedule task.
  3. This code does not have the change that sets the proxy since .NET automatically uses the system proxy by default in my testing
  4. Fixed improper disposing of WebClients that caused a TimeoutException when downloading multiple images.

If downloading from multiple regions, the code now does not force the wallpaper. Instead it is expected that user sets the wallpaper to be a slide show pointing to Bing Wallpaper\Year folder
Added support for hiding the console window. I use it when I run it as a schedule task.
@josueespinosa
Copy link
Owner

josueespinosa commented Jun 12, 2016

Hi @freaksterrao, sorry for the delayed response, I was on a business trip in Mexico. Everything looks great, but I need you to fix these things first:

  1. Build errors (the type or namespace name "market" could not be found)
  2. Remove gratuitous whitespace
  3. Follow One True Brace indent style

I've been meaning to push something like what you added for a while. The next step for me will probably be to create a GUI for the settings management and perhaps browsing the past 30 days of wallpapers available. Thanks for your help!

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

Successfully merging this pull request may close these issues.

3 participants