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

Barcode support #74 #93

Merged
merged 5 commits into from
Dec 1, 2014
Merged

Barcode support #74 #93

merged 5 commits into from
Dec 1, 2014

Conversation

ktind
Copy link
Contributor

@ktind ktind commented Nov 25, 2014

Fixed receiver leak
Moar tests

Fixed receiver leak
Moar tests
@ktind ktind changed the title Barcode support Barcode support #74 Nov 25, 2014
@@ -241,6 +240,7 @@ protected void onDestroy() {
super.onDestroy();
unregisterReceiver(mCGMStatusReceiver);
unregisterReceiver(mDeviceStatusReceiver);
unregisterReceiver(toastReceiver);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good call. Forgot about that :|

@rnpenguin
Copy link
Contributor

We also need to include references to zxing license. Probably both in the wiki and the open source dialog.

@ktind
Copy link
Contributor Author

ktind commented Nov 26, 2014

@rnpenguin I agree on the license - We have other licenses that we need to include as well. I'd like to get #91 in before updating the preference panel.

Joda Time, MongoDB, guava, zxing-android-minimal - Apache 2.0
SLF4J - MIT
Protobuf - BSD 3
MQTT will be Eclipse

@rnpenguin
Copy link
Contributor

👍

rnpenguin added a commit that referenced this pull request Dec 1, 2014
@rnpenguin rnpenguin merged commit 97219a1 into dev Dec 1, 2014
@rnpenguin rnpenguin deleted the wip/barcode#74 branch December 1, 2014 00:23
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.

4 participants