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

Added Possibility to choose the cameraId to be opened for decoding #71

Merged
merged 3 commits into from
Aug 19, 2016

Conversation

dlazaro66
Copy link
Owner

No description provided.

@@ -19,7 +19,8 @@ android {
}

dependencies {
compile 'com.dlazaro66.qrcodereaderview:qrcodereaderview:2.0.0'
//compile 'com.dlazaro66.qrcodereaderview:qrcodereaderview:2.0.0'
compile project (":qrcodereaderview")
Copy link
Owner Author

Choose a reason for hiding this comment

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

I'm finally setting this to link the local project instead of the remote maven repository

Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't think this is the right way to go. The sample project goal is to show how the library should be used. Using a compile project dependency is not something that library consumers will be able to do. Also using the real binary that is being deployed to maven will protect the sample project from unexpected errors.

@@ -54,6 +53,9 @@
void onQRCodeRead(String text, PointF[] points);
}

private static int CAMERA_FACING_BACK = Camera.CameraInfo.CAMERA_FACING_BACK;
Copy link
Collaborator

@saguinav saguinav Aug 19, 2016

Choose a reason for hiding this comment

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

Kill these constants

… example replacing the local project dependency.
@dlazaro66 dlazaro66 merged commit 9f40ff8 into master Aug 19, 2016
@dlazaro66 dlazaro66 deleted the manual-camera-id branch November 12, 2017 17:44
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.

2 participants