just1and0 / expo-push-notification-helper

Helpers/newChannel.js

import {Platform} from 'react-native';
import { Notifications } from 'expo';
import * as Permissions from 'expo-permissions';

const newChanneler = {
    // create a new channel
    newChannel: ( name, isSound )=>{
                  switch(isSound){
                      case false:
                       if (Platform.OS === 'android') {
                          Notifications.createChannelAndroidAsync(name, {
                            name: name,
                            sound: false,
                          });
                          return true;
                        }else{

else is not necessary here since all if branches return, consider removing it to reduce nesting and make code more readable.

                                alert('newChannel is not supported on IOS')

The alert UI element is often considered obtrusive and is generally only used as a temporary measure. Consider replacing it with another UI element.

                              }
                        break;

There is no return statement in this branch, but you do return something in other branches. Did you maybe miss it? If you do not want to return anything, consider adding return undefined; explicitly.

                   case true:
                       if (Platform.OS === 'android') {
                          Notifications.createChannelAndroidAsync(name, {
                            name: name,
                            sound: true,
                          });
                           return true;
                        }else{

else is not necessary here since all if branches return, consider removing it to reduce nesting and make code more readable.

                                alert('newChannel is not supported on IOS')
                              }
                        break;

There is no return statement in this branch, but you do return something in other branches. Did you maybe miss it? If you do not want to return anything, consider adding return undefined; explicitly.

                       default:
                             if (Platform.OS === 'android') {
                                Notifications.createChannelAndroidAsync(name, {
                                  name: name,
                                  sound: false,
                                });
                                 return true;
                              }else{

else is not necessary here since all if branches return, consider removing it to reduce nesting and make code more readable.

                                alert('newChannel is not supported on IOS')
                              }
                        break;

There is no return statement in this branch, but you do return something in other branches. Did you maybe miss it? If you do not want to return anything, consider adding return undefined; explicitly.

          }

    },


}

export default newChanneler;