-
Notifications
You must be signed in to change notification settings - Fork 2.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
InAppBrowserEvent declaration missing #3244
Closed
eyecatchup opened this issue
Nov 29, 2019
· 0 comments
· Fixed by #3258, wo-ist-henry/DriveWithMe#33, wo-ist-henry/DriveWithMe#34, wo-ist-henry/DriveWithMe#35 or wo-ist-henry/DriveWithMe#42
Closed
InAppBrowserEvent declaration missing #3244
eyecatchup opened this issue
Nov 29, 2019
· 0 comments
· Fixed by #3258, wo-ist-henry/DriveWithMe#33, wo-ist-henry/DriveWithMe#34, wo-ist-henry/DriveWithMe#35 or wo-ist-henry/DriveWithMe#42
Comments
akshaydwivedi29
added a commit
to akshaydwivedi29/ionic-native
that referenced
this issue
Dec 17, 2019
added data in Event interface for InApp Browser. fixes danielsogl#3244
danielsogl
pushed a commit
that referenced
this issue
Dec 27, 2019
added data in Event interface for InApp Browser. fixes #3244
This was referenced Jan 5, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I'm submitting a ...
[x] bug report
Current behavior:
The following code will throw a
Property 'data' does not exist on type 'InAppBrowserEvent'
errorThe
data
property is available since the postMessage implementation starting in InAppBrowser version 3.1.0.Expected behavior:
Property
data
needs to be defined here: https://github.com/ionic-team/ionic-native/blob/master/src/%40ionic-native/plugins/in-app-browser/index.ts#L128Something like:
Steps to reproduce:
Ionic code:
Ionic info: (run
ionic info
from a terminal/cmd prompt and paste output below):The text was updated successfully, but these errors were encountered: