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

Resolved merge conflicts #117

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

klausklemens
Copy link

  • merge conflicts resolved
  • the .NET version was changed to 4.6.1
  • removed Log folder from .gitignore because there is code in that folder

I did not look into cleaning anything else up. I think there are still some fragments from the earlier merge that should be cleaned up.

This Commit is a working version for me. I did not observe any major bugs.

Vincent added 7 commits July 9, 2021 18:00
+ merge conflicts resolved
+ the .NET version was changed to 4.6.1
+ removed Log folder from .gitignore because there is code in that folder

I did not look into cleaning anything else up. I think there are still some fragments from the earlier merge that should be cleaned up.

This Commit is a working version for me. I did not observe any major bugs.
It always proposes to update when it starts and i don't know how to make it stop
* CurrentWallpaperHolder.cs seems to sometimes not contain the image information. Now it always returns the current information in Properties.Settings because I am not familiar enough with the code to fix it properly
* WallpaperChanger.cs wouldn't always reset isRunning to 0 before a return
* RWC.cs the history page will now be recreated whenever the wallpaper changes (this might cause the program to hang for a few seconds whenever it comes up) and some blacklist calls wouldn't change the wallpaper
* RWC.resx just automatic changes i guess. no clue
- Old imgur links will work now
- Requests will now contain an over18 cookie with the value 1
- If an exception occurs while checking a wallpaper, it will now be rejected
* some inconsistencies among the settings files resolved
* added a TODO note to fix the auto updater
Starting without existing config files would cause a NullReferenceException because the Database tried to create a path to the database before the used Setting was initialized.
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.

1 participant