search

wix / react-native-notifications

React Native Notifications
MIT License
3.23k stars 763 forks source link

Is the Android import in MainApplication.java really needed? #855

Closed djibarian closed 2 years ago

djibarian commented 2 years ago

I wonder what's the purpose of the following import:

import com.wix.reactnativenotifications.RNNotificationsPackage;

in MainApplication.java. As stated by the docs this is needed, but I don't get any different results if I remove it.

Aure77 commented 2 years ago

I think it's useless. RN 0.6x+ auto link native modules. https://github.com/react-native-community/cli/blob/master/docs/autolinking.md

stale[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

stale[bot] commented 2 years ago

The issue has been closed for inactivity.