Update OneSignalPushAdapter.js [More Parameters Added for Android]

[mithleshjs]

Extra Parameters Added:

1. android_sound
2. small_icon
3. large_icon
4. big_picture
5. android_led_color
6. android_background_data

[flovilmart]

it would be nicer with a reduce:

post = ['android_background_data', ...].reduce( (post, key) {
  if (data[key]) {
    post[key] = data[key];
    delete data[key];
  }
  return post;
}, post);

[facebook-github-bot]

@mithleshjs updated the pull request.

[facebook-github-bot]

@mithleshjs updated the pull request.

[facebook-github-bot]

@mithleshjs updated the pull request.

[facebook-github-bot]

@mithleshjs updated the pull request.

[facebook-github-bot]

@mithleshjs updated the pull request.

[mithleshjs]

@flovilmart using reduce is causing build to fail and it overrides other params. I think it is best without a reduce.

[flovilmart]

What do you mean it overrides other params and build is failing?

[mithleshjs]

What I meant was that implementating reduce method as pointed by you is not working correctly.

[flovilmart]

You mean with the ... ?

[mithleshjs]

I was referring to this note by you.

post = ['android_background_data', ...].reduce( (post, key) {
if (data[key]) {
post[key] = data[key];
delete data[key];
}
return post;
}, post);

This method didn't work.

[flovilmart]

post = keys.reduce((post, key) => {
  if (data[key]) {
    post[key] = data[key];
    delete data[key];
  }
  return post;
}, post);