-
Notifications
You must be signed in to change notification settings - Fork 336
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
Use cityId for weather and add location picker when searching #315
Conversation
Hi @andrewmattie it's great to see this PR, it fixes a set of annoying and discouraging bugs. I'm not slavishly committed to Codacy's standards, it is mostly advice, but would you be able to take a look through their report into the PR and see what you think? Cheers |
@robinpaulson Sure, I'll go ahead and implement the fixes. They seem to be simple changes. |
OK, that's looking better now. The previous release appears to be working fine; a few days and I'll release 1.9 with the 3 queued PRs. |
Changed Detect Location to use cityId instead of name which allows for the correct city's weather to be displayed (#266).
Added dialog when searching for a location when there are multiple results (#259).