Added ability to import a barcode from a local image #51
Comments
|
Pretty cool! Are you up for adding some Robolectric tests for this new feature? Much of the app has automated tests, which helps prevent breakage and regressions. |
|
Sure. I'm not familiar with Robolectric, but I'll look into it. |
|
I'm quite interested in this feature too. I don't normally do Android development but I'm willing to try to help if useful |
|
I got a bunch of uni work dumped on me so I haven't had any time to work on this. It looks like I'll be clear beginning of December, so you can expect this finished around then. @TheLastProject thanks for the offer. I'm not sure GitHub would be very happy about mixing remote branches in one PR, but it's been a while since I've tried. If you can wait the two weeks, it's probably not worth the hassle of setting up Android Studio 😅 |
|
Okay, I'll just wait then :) |
|
I've been working on testing image importing by generating a barcode image, saving it and sending a fake pick intent response to the app. I have discovered, however, that the version of Robolectric used here doesn't actually implement image compression and is as such unable to save images to disk in a way that can read back. It seems that the feature was only implemented in v4.3.1. While the actual migration to v4 and up is not too difficult, it does, however, require compiling on SDK 28 or higher. Should I try to migrate the project to a newer version of Robolectric? |
|
I actually did a migration to Roboelectric 4 in my pull request in the hope it would fix things for me. It sadly didn't, but you may still want to take a look at brarcher/loyalty-card-locker@0a9611e to save yourself some time if you end up going that path |
|
The Play Store has a target SDK requirement which updates each year. As of Nov 1 this app will need to compile against SDK 28, else an update cannot be pushed to the Play Store. https://support.google.com/googleplay/android-developer/answer/113469#targetsdk If you would be so kind as to update it to 28 (or 29) that would be much appreciated. I'm also fine with updating the version of Robolectric being used to the latest. |
|
To follow-up, brarcher/loyalty-card-locker#334 recently was accepted which updates the SDK version compiled against to 29 and updates Robolectric to 4.0.2. If you are willing to upgrade the version of Robolectric used from 4.0.2 to 4.3.1 go for it. I've not looked into it; hopefully nothing that the existing tests use is deprecated and removed between those two versions. If you run into snags, let me know. |
|
@franga2000 Are you still interested in working on this? Otherwise I'll see if I can integrate this into Catima myself. |
|
I just got the app, and can't migrate from my old app (Fidme) to Catima quickly. |
|
Fixed in c93b752, going to give the translators a few days to translate stuff and then I'll push out a new release. |
Monday Oct 14, 2019 at 18:43 GMT
Originally opened as brarcher/loyalty-card-locker#317
Adds an "Import from image" button next to "Capture card" that does exactly what was described in #248 - allows the user to pick an image from their device that contains the barcode they want to add.
Signed-off-by:
Miha Frangež<[email protected]>This change is
franga2000 included the following code: https://github.com/brarcher/loyalty-card-locker/pull/317/commits
The text was updated successfully, but these errors were encountered: