Inconsistent data return from checkIfHasSMSPermission

[alokpant]

Describe the bug
At the moment, checkIfHasSMSPermission returns multiple data types:

1. Object (https://github.com/maniac-tech/react-native-expo-read-sms/blob/master/index.js#L56)
2. Boolean (https://github.com/maniac-tech/react-native-expo-read-sms/blob/master/index.js#L54, https://github.com/maniac-tech/react-native-expo-read-sms/blob/master/index.js#L44)

As per documentation, it says that it returns an object:
https://github.com/maniac-tech/react-native-expo-read-sms/blob/master/README.md?plain=1#L49

To Reproduce
Steps to reproduce the behavior:

1. Call the method

const permissions = await checkIfHasSMSPermission();

2. it returns either true or { hasReceiveSmsPermission: true/false, hasReadSmsPermission: true/false }

Expected behavior
It should always return an object or boolean.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

• OS: [e.g. iOS]
• Browser [e.g. chrome, safari]
• Version [e.g. 22]

Smartphone (please complete the following information):

• Device: Oneplus 8T
• OS: Android
• Browser (compiled app using Android Studio + Expo
• Version OxygenOS 14.0

Additional context
Add any other context about the problem here.

[alokpant]

Also created a PR on the fix: #76

In the PR, i went with returning the boolean, since we can use return value for updating multiple states. This would help with multiple cases, for example to either for READ_SMS or RECEIVE_SMS.

[maniac-tech]

Also created a PR on the fix: #76

In the PR, i went with returning the boolean, since we can use return value for updating multiple states. This would help with multiple cases, for example to either for READ_SMS or RECEIVE_SMS.

Appreciate the effort @alokpant .

Will review the PR this week.

[maniac-tech]

Haven been able to review it so far.
Planning to review this by tomorrow.