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

Support Capacitor 6.0 #22

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

Support Capacitor 6.0 #22

wants to merge 4 commits into from

Conversation

afilp
Copy link

@afilp afilp commented Apr 22, 2024

No description provided.

@mantosh59
Copy link

If everything is okay then please merge the PR asap.

@fatihtkale
Copy link

Can we get this merged?

@fatihtkale
Copy link

image

I get this issue when building to iOS.

@fatihtkale
Copy link

fatihtkale commented May 13, 2024

also in
CapacitorDocumentScanner.podspec
s.homepage cannot be null xcode wont build if its null for some reason?

@emelampianakis
Copy link

image

I get this issue when building to iOS.

Hello @fatihtkale,
it builds fine on my capacitor 6 project, can you make sure your pods are up to date?
my pod version is 1.14.3.

@fatihtkale
Copy link

image
I get this issue when building to iOS.

Hello @fatihtkale, it builds fine on my capacitor 6 project, can you make sure your pods are up to date? my pod version is 1.14.3.

Hey my friend im running 1.15.2
i actually got it to work by manually taking the files and putting them together

@fatihtkale
Copy link

the issue with DocScanner was because it couldnt find it and thats a me issue i did a quick and dirty solution and just made a new git repo with the required files and it works now
https://github.com/fatihtkale/docusign here

@fatihtkale
Copy link

i guess you could also do yarn add WebsiteBeaver/capacitor-document-scanner#22/head to update the package but what i did was just faster for me and easier since pod itself gave me a "s.homepage" undefined error as shown above

@vsatya-work
Copy link

@dmarcs Could you please upgrade the plugin to support Capacitor 6.0?

@nesgarbo
Copy link

nesgarbo commented Jun 8, 2024

https://github.com/fatihtkale/docusign

Could you please create de repo or put it as public pls.

Thx

@Martijn0405
Copy link

@afilp What is the ETA for this merge?

@afilp
Copy link
Author

afilp commented Jun 12, 2024

@Martijn0405 I cannot say as I am not the maintainer of this repo, I just submitted a possible fix. Thanks.

@fatihtkale
Copy link

The repo is now public for anyone wanting to use it

@mantosh59
Copy link

Have created plugin which will support latest capacitor version. Check this out.

https://www.npmjs.com/package/mdocument-scanner

@Martijn0405
Copy link

Merge please!

@therealabdi2
Copy link

any eta on this PR?

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

Successfully merging this pull request may close these issues.

8 participants