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

Fixed inconsistency geopoint widgets button name #1986

Merged
merged 2 commits into from
Mar 12, 2018
Merged

Fixed inconsistency geopoint widgets button name #1986

merged 2 commits into from
Mar 12, 2018

Conversation

isabelcosta
Copy link
Contributor

What has been done to verify that this works as intended?

I've been experimenting the Geopoint widgets, but I'm having trouble reproducing the issue problem. I picked up where the previous contributor left.
I ran the application on an emulator - Nexus 5 API 26 with Android 8.0.

Why is this the best possible solution? Were any other approaches considered?

The solution appeared to have the bug in the logic for the lack of available data. No.

Are there any risks to merging this code? If so, what are they?

There are no risks by merging this code.

Do we need any specific form for testing your changes? If so, please attach one.

I'm using All Widgets form.

@lognaturel
Copy link
Member

Thanks, @isabelcosta! Works as expected for me. Perhaps the emulator didn't have a default value set so you were never getting a geopoint reading?

Probably no need for much verification but I'm tagging as needs testing to follow the typical QA process. No need to check more than one device/Android version

@isabelcosta
Copy link
Contributor Author

Thanks for the review @lognaturel!
I think I got the GeoPoint reading. I got this result:

image

@lognaturel
Copy link
Member

👍 The one you edited the button label for is the question before in All Widgets — geopoint with no appearance.

@mmarciniak90
Copy link
Contributor

Tested with success!

Verified on Android: 4.1, 4.2, 4.4, 5.1, 6.0, 7.0, 8.0.

Verified cases:

  • Adding location
  • Remove response
  • OSM and Google Maps

@opendatakit-bot unlabel "needs testing"
@opendatakit-bot label "behavior verified"

@lognaturel lognaturel merged commit 2b1a3ee into getodk:master Mar 12, 2018
shobhitagarwal1612 pushed a commit to shobhitagarwal1612/collect that referenced this pull request May 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inconsistency between Geopoint widgets' button name
4 participants