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

#5 Incompatible with Gradle Version 9 - Upgrade react native 72 #6

Merged
merged 5 commits into from
Aug 24, 2023

Conversation

R4DIC4L
Copy link
Contributor

@R4DIC4L R4DIC4L commented Aug 22, 2023

This fixed #5 where the lastest gradle version is not supported because of below error.

Could not set unknown property 'classifier' for task ':dashdoc_react-native-system-sounds:androidSourcesJar' of type org.gradle.api.tasks.bundling.Jar.

I have upgraded both react-native-system-sounds and the example to use react-native v0.72.4. I have used upgrade helper to aid me with the upgrade: https://react-native-community.github.io/upgrade-helper/?from=0.63.5&to=0.72.4.

I have tested the android application using the example app and yarn as instructed in the README. I confirm that the sounds are working.

I don't have an iOS device available at the moment, but will have one to test this also on iOS in the following days. Any help to test this on iOS would also be appreciated.

Can you take a look at this PR?

@aouaki aouaki requested a review from asadwan August 22, 2023 14:00
package.json Outdated
@@ -50,11 +50,11 @@
"react-native": "*"
},
"devDependencies": {
"@types/react-native": "^0.63.37",
"@types/react-native": "^0.72.2",
Copy link
Contributor

Choose a reason for hiding this comment

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

Starting from RN 71, types are built-in. You can remove this dep

Copy link
Contributor Author

@R4DIC4L R4DIC4L Aug 23, 2023

Choose a reason for hiding this comment

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

Should be removed now, also from example app.

@asadwan
Copy link
Contributor

asadwan commented Aug 23, 2023

Hi R4DIC4L, thanks for the PR.
I tested iOS and I confirm it's working

dashdoc#5 remove unnecessary dependency for @types/react-native package for v0.72
dashdoc#5 remove unnecessary dependency for @types/react-native package for v0.72 from example app
@asadwan asadwan merged commit 242abab into dashdoc:main Aug 24, 2023
@R4DIC4L
Copy link
Contributor Author

R4DIC4L commented Aug 24, 2023

Any ETA for when this can be available in a future npm release?

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.

Incompatible with Gradle Version 9
2 participants