Skip to content
This repository has been archived by the owner on Oct 10, 2024. It is now read-only.

Update/capacitor v6 #302

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

pablo-perez-seagull-com

No description provided.

@julienkermarec
Copy link

+1 validated

@julienkermarec
Copy link

@thegnuu possible to merge this PR for capacitor 6 ?

@thegnuu
Copy link
Collaborator

thegnuu commented Jul 30, 2024

@julienkermarec we deprecated this library for multiple reasons and I don't really have time to work on this at the moment.
Unfortunately I cannot even merge this PR since the commits are not signed and this is required by the capacitor-community org.

If someone is willing to create a signed PR and properly test it out, I can merge it and create a beta release or so since I will not have time to test everything.

@reslear
Copy link

reslear commented Sep 17, 2024

@umeshsarkar
Copy link

umeshsarkar commented Sep 17, 2024

@reslear I would not recommend archiving the project.

https://capacitorjs.com/docs/apis/barcode-scanner : the library is still under development and currently offers only one API scanBarcode(). Additionally, the current version does not allow for customizing the scanner's UI or scanning multiple codes at once.

https://capawesome.io/plugins/mlkit/barcode-scanning/: this library can not properly scan the ITF, Code 128, and Code 39 (known issue).

Both issues have not provided a solution for the barcode scanning problem. It makes those libraries sort of unreliable.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants