Closed runelaenen closed 9 years ago
I also wrote a small update script (just notify the user if an update was available) but reverted it since I think it should be in a different pull request. It also wasn't perfect
Ready for merge?
It builds, so I'd say it's OK. I'll fix the resource issues and change the login system a bit to a more modular approach. It prevents duplication of code everywhere.
Oh, and @runelaenen, if you want to work on two pull requests at the same time, create different branches. That prevents commitcluttering like this. You could name them like 'feature-[name]'. That's following the conventions in this repo as well.
All right, I've also forgot some things in the config.vb file (like setting the default values) since I only saw that I should've done that way later (and I edited a lot more in my project so I'm afraid I can't push it)
The timer only polls the server when the process has quit instead of every 5 minutes. Also added a link to OpenRCT.net on the login window.