Mmmh should work, I double checked there is an implementation https://github.com/ionic-team/ionic-native/blob/v5/src/%40ionic-native/plugins/onesignal/index.ts
Did you also npm install @ionic-native/core/beta
?
P.S.: I just link this issue because I guess there are related Onesignal not work in ionic 4