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

Fix #1759 multimeter record bug fixed #1761

Merged
merged 2 commits into from Jun 1, 2019

Conversation

neel1998
Copy link
Member

Fixes #1759

Changes: Bug fixed, pause button replaced with stop button, csv file created on clicking stop button.

Screenshot/s for the changes:
20190531_185414

Checklist: [Please tick following check boxes with [x] if the respective task is completed]

  • I have used resources from strings.xml, dimens.xml and colors.xml without hard-coding them
  • 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:
multimeter_log.zip

@CloudyPadmal kindly review

@neel1998
Copy link
Member Author

@CloudyPadmal it would be great if you can merge this so that i can start working on #1762

@CloudyPadmal CloudyPadmal added the Status: Review Required Requested reviews from peers and maintainers label Jun 1, 2019
@neel1998
Copy link
Member Author

neel1998 commented Jun 1, 2019

@CloudyPadmal i have changed the stop icon. I think this can be merged now.
Screenshot_20190601-140213

@CloudyPadmal CloudyPadmal merged commit 9c2313b into fossasia:development Jun 1, 2019
@CloudyPadmal CloudyPadmal removed the Status: Review Required Requested reviews from peers and maintainers label Jun 1, 2019
neel1998 added a commit to neel1998/pslab-android that referenced this pull request Jul 30, 2019
* multimeter record bug fixed

* stop button icon changed
@neel1998 neel1998 deleted the multimeter_log branch August 27, 2019 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

App crashes on clicking export data option of multimeter
2 participants