-
Notifications
You must be signed in to change notification settings - Fork 9.8k
Android and iOS BarcodeDetector options #677
Android and iOS BarcodeDetector options #677
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
"pronunciation", barcode.getContactInfo().getName().getPronunciation()); | ||
name.put("suffix", barcode.getContactInfo().getName().getSuffix()); | ||
Map<String, Object> name = new HashMap<>(); | ||
if (barcode.getContactInfo().getName() != null) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe save barcode.getContactInfo().getName()
in an instance variable to make the code below simpler?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might also be nice for the code below...
@@ -1,7 +1,7 @@ | |||
name: firebase_ml_vision | |||
description: Flutter plugin for Google ML Vision for Firebase, an SDK that brings Google's machine | |||
learning expertise to Android and iOS apps in a powerful yet easy-to-use package. | |||
version: 0.0.2 | |||
version: 0.0.3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The change log says this is 0.1.0?
## 0.1.0 | ||
|
||
* **Breaking Change** Add Barcode, Face, and Label on-device detectors. | ||
* Remove close method. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was that removed in a previous PR? Doesn't seem to be included in this one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yea, I snuck in the version upgrade so you would only need to review 2 PRs not 3. :)
No description provided.