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

feat(android): use fresco 3.2.0 in order to fix animation for gifs #15583

Merged
merged 1 commit into from
Feb 7, 2025

Conversation

Calinteodor
Copy link
Contributor

No description provided.

@Calinteodor Calinteodor assigned saghul and unassigned saghul Feb 7, 2025
@@ -69,10 +69,29 @@ allprojects {
}
}

// Due to a dependency conflict between React Native and the Fresco library used by GiphySDK,
// GIFs appear as static images instead of animating
// https://github.com/Giphy/giphy-react-native-sdk/commit/7fe466ed6fddfaec95f9cbc959d33bd75ad8f900
Copy link
Member

Choose a reason for hiding this comment

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

Isn't that the solution for RN 0.76? For below it says:

Developers using our SDK with React Native versions from 0.73 up to but not including 0.76 can resolve this conflict
by adding the following block to the android/build.gradle file in the :app module,
right before the dependencies section:

configurations.all {
    resolutionStrategy {
        forcedModules = ['com.giphy.sdk:ui:2.3.13']
    }
}

Copy link
Contributor Author

@Calinteodor Calinteodor Feb 7, 2025

Choose a reason for hiding this comment

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

I know, but no. That doesn't work. I think maybe they are mistaken, at least starting with rn 0.75.4 that doesn't work.

@Calinteodor Calinteodor merged commit 75b4049 into jitsi:master Feb 7, 2025
10 checks passed
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.

2 participants