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

Upgrade to Capacitor 6 #355

Merged
merged 2 commits into from
Apr 22, 2024
Merged

Upgrade to Capacitor 6 #355

merged 2 commits into from
Apr 22, 2024

Conversation

KerimM
Copy link
Contributor

@KerimM KerimM commented Apr 20, 2024

closes #354
closes #356

package.json Outdated
@@ -1,6 +1,6 @@
{
"name": "@codetrix-studio/capacitor-google-auth",
"version": "3.4.0-rc.0",
"version": "3.5.0-rc.0",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thi is already cap6 verision, please change to 3.4.0-rc.1

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@reslear reslear self-assigned this Apr 21, 2024
@reslear
Copy link
Collaborator

reslear commented Apr 21, 2024

I'll have to remember to return #345.

@KerimM
Copy link
Contributor Author

KerimM commented Apr 22, 2024

I'll have to remember to return #345.

done

@reslear
Copy link
Collaborator

reslear commented Apr 22, 2024

@KerimM thank you 🔥

@reslear reslear merged commit abdf3aa into CodetrixStudio:master Apr 22, 2024
@ManuelCostanzo
Copy link

Hey, sorry I have a question, why are you doing

CAP_PLUGIN(GoogleAuth, "GoogleAuth")

when that macro needs three params?

@reslear
Copy link
Collaborator

reslear commented Apr 23, 2024

hi @ManuelCostanzo That's not right, can you create a revert PR?

@KerimM
Copy link
Contributor Author

KerimM commented Apr 24, 2024

Hey, sorry I have a question, why are you doing

CAP_PLUGIN(GoogleAuth, "GoogleAuth")

when that macro needs three params?

I follow the migration guide

Breaking changes in code
iOS
Remove removeAllListeners method
If you plugin had CAP_PLUGIN_METHOD(removeAllListeners, CAPPluginReturnPromise); in the .m file, it can be removed now, the method is now available for all plugins without defining it.

@reslear
Copy link
Collaborator

reslear commented Apr 24, 2024

@KerimM maybe need remove only core method removeAllListeners ?)

like:

https://github.com/Cap-go/capacitor-updater/blob/3a8da63fe2bc0b4ad2df7268fb057a2b204e01c1/ios/Plugin/CapacitorUpdaterPlugin.m#L29

@gasci
Copy link

gasci commented Apr 28, 2024

Is there an update regarding this topic? 3.4.0-rc.1 fails @KerimM

@reslear
Copy link
Collaborator

reslear commented Apr 29, 2024

hi @gasci check please v3.4.0-rc.2

@gasci
Copy link

gasci commented Apr 29, 2024

For some reason rc.2 isn't available. Am I doing something wrong?
Screenshot 2024-04-29 at 11 56 27

The most recent version is the one here for some reason.
Screenshot 2024-04-29 at 14 33 38

Edit
The method below worked:
npm install "CodetrixStudio/CapacitorGoogleAuth#v3.4.0-rc.2"

@gasci
Copy link

gasci commented Apr 29, 2024

I confirm that v3.4.0-rc.2 works with capacitor version 6.0.0. Could you release v3.4.0-rc.2 as version 6.0.0 for this plugin to align with the capacitor version, please? @reslear

Other community plugins mostly follow this convention.

@reslear
Copy link
Collaborator

reslear commented Apr 30, 2024

done, published 3.4.0-rc.3

@mburger81
Copy link

When will we see this plugin as stable release for capacitor@6, thx you a lot for your work

@gasci
Copy link

gasci commented May 31, 2024

I believe it is breaking on M1 computers and there a few major bugs that prevent this plugin to be stable for now. @mburger81

I created issues as I detect them:
#363

@mburger81
Copy link

Ok but this issue has as source the @capacitor@6 update? @gasci if yes thats obviously bad, thx for your response

@gasci
Copy link

gasci commented May 31, 2024

I believe that is the case. It started to occur after my capacitor6 upgrade for this plugin

@mburger81
Copy link

@gasci it seems you have fixed the problem here
#363 (comment)

so we should be able to have a stable release?

@gasci
Copy link

gasci commented Jul 5, 2024

I would say so. Yes, calling the init methods before others fixes the issue. @mburger81

@mburger81
Copy link

@gasci can we have a official release for capacitor@6? I mean at the end of august you can't upload to Gogole Play Store if you don't use Android14+ which means for use to upgrade to capacitor@6.

@gasci
Copy link

gasci commented Aug 5, 2024

Hi, I don't think I have the permissions to do that. Please reach out to the original maintainers.

@mburger81
Copy link

Oh yea thats true, sorry @gasci

@KerimM @reslear can you please do a official release without the -rc.x ?

@reslear
Copy link
Collaborator

reslear commented Aug 5, 2024

@mburger81 you have to test it first

@gasci
Copy link

gasci commented Aug 6, 2024

I tested this version and I am running it without any problems. (3.4.0-rc.4)

But please see this issue if you are having issues on tablets and M1 computers:
#363

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.

Capacitor 6 support Compatibility with capacitor 6
5 participants