search

parse-server-modules / parse-server-onesignal-push-adapter

OneSignal push adapter for parse-server
MIT License
32 stars 28 forks source link

Add support for push time #22

Open yingliangzhang opened 8 years ago

yingliangzhang commented 8 years ago

This adds basic support for push time for sending scheduled notifications. Would appreciate any feedback around how best to capture this in the test cases.

codecov-io commented 8 years ago

Current coverage is 78.33% (diff: 66.66%)

Merging #22 into master will decrease coverage by 0.61%

@@             master        #22   diff @@
==========================================
  Files             1          1          
  Lines           114        120     +6   
  Methods           9          9          
  Messages          0          0          
  Branches         18         20     +2   
==========================================
+ Hits             90         94     +4   
- Misses           24         26     +2   
  Partials          0          0

Powered by Codecov. Last update b1d0539...04d485a

yingliangzhang commented 8 years ago

Would appreciate any feedback on this. Thanks!

juliusbtesh commented 7 years ago

This was not working for me. I added 5 minute to the push time but the push went right away. But the weird part is the logs showed that the 5 minutes were added as a scheduled push but the actual pushStatus said the current time