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 crash in Import/Export settings #2192

Merged
merged 1 commit into from May 16, 2018

Conversation

grzesiek2010
Copy link
Member

@grzesiek2010 grzesiek2010 commented May 14, 2018

Closes #2191

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

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

I just removed not needed changes added in #2140 probably by mistake since we don't need them.

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

No.

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

No.

Does this change require updates to documentation? If so, please file an issue here and include the link below.

No.

Before submitting this PR, please make sure you have:

  • run ./gradlew pmd checkstyle lint findbugs and confirmed all checks still pass.
  • verified that any new UI elements use theme colors so that they work with both light and dark themes.

@mmarciniak90
Copy link
Contributor

Tested with success!

Verified on Android: 4.1, 4.2, 4.4, 5.1, 6.0, 8.1

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

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.

None yet

4 participants