Skip to content

Conversation

@erisu
Copy link

@erisu erisu commented Feb 4, 2025

Description

Appears that the front-end handler does not get the notification payload when tapping on the notification toast while app is in the background.

Related Issue

n/a

Motivation and Context

Deliver the notification payload to the front-end handler when clicking on notification toast while app is in the background.

How Has This Been Tested?

Built app with changes & tested follow case:

  • Drop payload when app is open
  • Drop payload when app is closed
  • Drop payload when app is in background

Verify that notification event handler in front-end is triggered with payload.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@erisu erisu force-pushed the fix/ios-notification-payload-to-front-when-in-background branch from d8bc984 to 53365c6 Compare February 4, 2025 06:56
@erisu erisu marked this pull request as draft February 4, 2025 12:55
@EtienneLem
Copy link

Hey @erisu 👋🏼,

We have users reporting something quite similar than what this PR seems to be solving, which is how we found the PR.

We’re more than comfortable using a fork of ours and applying this change; so I’m not necessarily asking whether this can be merged or not. I’m curious about the Draft state of the PR though. Do you recall if there was something not working with this branch and/or not finished?

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants