bug: PushNotificationSchema on ios has 'extra' instead of 'data'

[moxaj]

Bug Report

Capacitor Version

Latest Dependencies:

@capacitor/cli: 2.4.7 @capacitor/core: 2.4.7 @capacitor/android: 2.4.7 @capacitor/ios: 2.4.7

Installed Dependencies:

@capacitor/cli: 3.0.0-beta.6 @capacitor/android: 3.0.0-beta.6 @capacitor/ios: 3.0.0-rc.0 @capacitor/core: 3.0.0-beta.6

Platform(s)

ios

Current Behavior

The PushNotificationSchema object exposed in the pushNotificationActionPerformed callback has the associated key-value pairs in its extra field.

Expected Behavior

The key-value pairs should be assigned to the data field, as in android.

Code Reproduction

See:

• ios implementation at: https://github.com/ionic-team/capacitor-plugins/blob/2a83fcb536cdfc5b601f363212353201de40ca5b/push-notifications/ios/Plugin/PushNotificationsHandler.swift#L86
• android implementation at: https://github.com/ionic-team/capacitor-plugins/blob/2a83fcb536cdfc5b601f363212353201de40ca5b/push-notifications/android/src/main/java/com/capacitorjs/plugins/pushnotifications/PushNotificationsPlugin.java#L72

(The other fields seem somewhat inconsistent as well)

Other Technical Details

npm --version output: 6.14.6

node --version output: v12.18.3

pod --version output (iOS issues only): 1.10.1

[vitorlopez]

Please merge this 😄

[vitorlopez]

Thank you so much.

Do you know when this is going to be pushed to npm, or for that matter when a new capacitor RC is going to be pushed? I see there's a lot of fixes already merged to main.

Thanks again.

[ionitron-bot[bot]]

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of the plugin, please create a new issue and ensure the template is fully filled out.