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

Fixes #2032 Fixes crash which occurred in the oscilloscope #2102

Merged
merged 1 commit into from
Apr 23, 2020
Merged

Fixes #2032 Fixes crash which occurred in the oscilloscope #2102

merged 1 commit into from
Apr 23, 2020

Conversation

marcnause
Copy link
Contributor

@marcnause marcnause commented Feb 4, 2020

Fixes #2032

Changes: Use join method of Apache StringUtils instead of the join method of the Android String class. The join method was added to the String class of Android in API level 26, but the minimum API level of the app is 23.

Checklist:

  • No modifications done at the end of resource files strings.xml, dimens.xml or colors.xml
  • I have reformatted code in every file included in this PR [CTRL+ALT+L]
  • My code does not contain any extra lines or extra spaces
  • I have requested reviews from other members

APK for testing: app-fdroid-debug.zip

@marcnause marcnause marked this pull request as ready for review February 4, 2020 18:56
@CloudyPadmal CloudyPadmal added Fix Solution to an existing issue in app Status: Review Required Requested reviews from peers and maintainers labels Feb 4, 2020
@cweitat cweitat merged commit b35835b into fossasia:development Apr 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Solution to an existing issue in app Status: Review Required Requested reviews from peers and maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Oscilloscope: App crashes as soon as record button is clicked while using built-in mic.
3 participants