search

ronny3050 / phone-notification-mirror

Displays and alerts mirror users on new phone notifications
61 stars 15 forks source link

Wrong example for config.js.... #10

Closed cowboysdude closed 7 years ago

cowboysdude commented 7 years ago

You are using this as an example ->

modules: [ { module: 'phone_notification', position: 'bottom_right', header: 'Phone Notifications', config:{ accessToken: 'YOUR_ACCESS_TOKEN', numberOfNotifications: 5, displayNotificationIcon: true, displayMessage: true, displayCount: false, alert: false, fade: true, maxCharacters: 50 }, } ]

It's confusing new users... the module array already starts at the top.. people are trying to add your config example as written and it's not going to work ;)

Please change example to this ->

{
        module: 'phone_notification',
        position: 'bottom_right',
        header: 'Phone Notifications',
        config:{
            accessToken: 'YOUR_ACCESS_TOKEN',
            numberOfNotifications: 5,
            displayNotificationIcon: true,
            displayMessage: true,
            displayCount: false,
            alert: false,
            fade: true,
            maxCharacters: 50
        }
},
ronny3050 commented 7 years ago

Thank you so much for the correction! 👍 :)