search

capacitor-community / image-to-text

Capacitor Plugin for image to text processing (OCR)
Other
28 stars 10 forks source link

feat: accept Base64 too #4

Closed ludufre closed 6 months ago

ludufre commented 1 year ago

Hi, I added support to Data Url (base64 image) as param.

Captura de Tela 2023-07-06 às 14 06 25 Captura de Tela 2023-07-06 às 14 06 32

jroeglin commented 1 year ago

We are interested in this feature please merge.

dtarnawsky commented 1 year ago

Waiting on fixes before considering merging.

ludufre commented 8 months ago

Hi @dtarnawsky I ended up forgetting about this PR. Are you referring to any fixes I need to do? I'm lost...

ludufre commented 6 months ago

Hi @dtarnawsky Is there anything I can do to make this functionality accepted? Or is it really not interesting?

dtarnawsky commented 6 months ago

@ludufre , It looks like the review comments were addressed but there wasn't a reply on this thread so I assumed it wasn't addressed.

I'll do some quick testing and will include this PR in a v6 release.

dtarnawsky commented 6 months ago

@ludufre Are you able to resolve the conflicts on the README.md?