Skip to content
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

Require Moengage Module integration for Push Notifications, In App Notifications and events #1773

Closed
mehsaandev opened this issue Dec 16, 2024 · 4 comments · Fixed by #1787
Closed
Assignees

Comments

@mehsaandev
Copy link
Contributor

Require the following Modules from Moengage:

  • In-App Notification Banners
  • Push Notifications
  • Deep Linking to navigate on a Specific Screen when taping Notification.
  • Handle Moengage Events
@kmahmood74
Copy link
Collaborator

@mehsaandev what does this mean - Deep Linking to navigate on a Specific Screen when taping Notification. what will the deep link look like and which screen should it go with what payload? Please be specific with examples instead of a super generic statement

@mehsaandev
Copy link
Contributor Author

mehsaandev commented Dec 16, 2024

  • Deep Linking for Ensemble:
    Following is the dashboard section in Moengage for deep linking while creating notification campaign.
    I can add key-value pairs for deep linking in the notifications.
    For navigation Please consider screen key to get the target screen to navigate on.

image

CC: @TheNoumanDev

@kmahmood74
Copy link
Collaborator

@mehsaandev have you confirmed with the customer that they will be creating special notifications with the screen names which means that their existing notification payloads will not work anymore? Are we sure we want to impose this?

@mehsaandev
Copy link
Contributor Author

@mehsaandev have you confirmed with the customer that they will be creating special notifications with the screen names which means that their existing notification payloads will not work anymore? Are we sure we want to impose this?

@kmahmood74 Yes I've asked the customer and they are totally fine with this solution.

@TheNoumanDev TheNoumanDev linked a pull request Dec 24, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants