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

Compatibility with Android SDK 34 #56

Closed
DupuyNicolas opened this issue Nov 23, 2023 · 10 comments
Closed

Compatibility with Android SDK 34 #56

DupuyNicolas opened this issue Nov 23, 2023 · 10 comments

Comments

@DupuyNicolas
Copy link

Describe the bug
The plugin won't compile when targeting Android SDK 34.
This is because of a change on the onFling method overriden by OnSwipeTouchListener
Unfortunately, this method now takes a nullable "downEvent: motionEvent", so it must be updated accordingly in the method parameters, and must be checked or asserted below.

To Reproduce
Steps to reproduce the behavior:

  1. Target SDK 34 on Android application
  2. Build the app
  3. See error in OnSwiperTouchListener

Expected behavior
The plugin should compile with SDK 34.

@gustavomodena
Copy link

Any update? Im facing the same issue

@jepiqueau
Copy link
Collaborator

@DupuyNicolas @gustavomodena it will not be done before Capacitor 6 is released as i am not sure how Capacitor 5 is compatible with API34

@andrerds
Copy link

andrerds commented Feb 9, 2024

"Note: I encountered an issue after migrating to Capacitor 6, requiring me to download the latest version of Android Studio. After making changes to the marked lines, I successfully compiled the code.

In the OnSwipeTouchListener class and the 'onFling' function:

override fun onFling(
    downEvent: MotionEvent?,  // Change made in this line
    moveEvent: MotionEvent,
    velocityX: Float,
    velocityY: Float
): Boolean {
     var diffX = moveEvent.x.minus(downEvent!!.x)  // Change made in this line

@jepiqueau
Copy link
Collaborator

@andrerds thanks for this it will be helpful to know when i will do the migration when ionic will have release Capacitor6

@JulienLecoq
Copy link

Ionic has released Capacitor 6 since a few weeks now

@jepiqueau
Copy link
Collaborator

@andrerds @JulienLecoq @DupuyNicolas @gustavomodena Please read the Note i just published on the README file of github.com/capacitor-community/photoviewer/README.md.

@andrerds
Copy link

@andrerds @JulienLecoq @DupuyNicolas @gustavomodena Please read the Note i just published on the README file of github.com/capacitor-community/photoviewer/README.md.

Hello, my friend. Thanks a lot for your contribution, you are fantastic, congratulations. I hope everything goes well for you and your family. Family is a priority. Thank you for everything!

@shiv19
Copy link
Contributor

shiv19 commented May 16, 2024

@jepiqueau Thank you for all the years of hard work you have put in for this community.
I have nothing but admiration for your contribution to this eco system.
Will you be moving any more plugins into the @capacitor-community namespace? Particularly the video-player.
I'm fairly new to the ionic community and am looking to contribute to the plugins.

@jepiqueau
Copy link
Collaborator

@andrerds @shiv19 Thanks for your words, they go to my heart. @shiv19 the best if you want to contribute to the capacitor community is to talk to Mike Hartington [email protected]

dootix-alberto added a commit to dootixsa/photoviewer that referenced this issue Jul 9, 2024
eric-horodyski added a commit to eric-horodyski/capacitor-community-photoviewer that referenced this issue Aug 20, 2024
@robingenz
Copy link
Member

This should be fixed in #63.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants