-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
Update to capacitor 6 #58
Update to capacitor 6 #58
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just one small question inline. Sorry for keeping you waiting and thanks for the effort :)
@@ -1,5 +1,5 @@ | |||
|
|||
<manifest xmlns:android="http://schemas.android.com/apk/res/android" | |||
package="com.timozacherl.plugins.CapSiriShortcuts"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Has this been removed by the automatic migration?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, package name has been moved to the build.gradle automatically: https://github.com/lovetodream/capacitor-plugin-siri-shortcuts/pull/58/files/9e2d636f2e98dc2e01ae2aec0ce24ad58ce01bb8#diff-197b190e4a3512994d2cebed8aff5479ff88e136b8cc7a4b148ec9c3945bd65aR21
One additional note, can you please add a disclaimer for v6 to the breaking changes section in the readme as done with all the previous versions? |
💥 Breaking Changes
Hello, I glad inform to you that the README.md was updated following the same structure that was used in previous versions of the software. Cheers! |
Looks good, thank you for doing this! |
Hello,
Following https://capacitorjs.com/docs/updating/plugins/6-0, I update plugin to Capacitor 6.
Cheers!