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

Push notification body gets truncated #2857

Closed
1 task done
jamilsaadeh97 opened this issue Jun 20, 2024 · 6 comments · Fixed by #2856
Closed
1 task done

Push notification body gets truncated #2857

jamilsaadeh97 opened this issue Jun 20, 2024 · 6 comments · Fixed by #2856
Labels
feature-request Request a new feature Push Notifications Related to the push notifications category

Comments

@jamilsaadeh97
Copy link
Contributor

Before opening, please confirm:

Language and Async Model

Not applicable

Amplify Categories

Notifications

Gradle script dependencies

Not applicable

Environment information

Not applicable

Please include any relevant guides or documentation you're referencing

No response

Describe the bug

Notifications are being truncated when the body exceeds ~90 characters. The reason is that "long" notifications are not being handled here by using the BigTextStyle.

I'm using Amplify Flutter and I can't override these properties in my app.

I've opened #2856 to fix this

Thank you!

Reproduction steps (if applicable)

  1. Put app in the background
  2. Send a notification with body: "This is a long push notifications message that's supposed to be completely shown on the user device"

Code Snippet

// Put your code below this line.

Log output

// Put your logs below this line


amplifyconfiguration.json

No response

GraphQL Schema

// Put your schema below this line

Additional information and screenshots

No response

@vincetran
Copy link
Member

Hi @jamilsaadeh97 Thanks for the report and the PR! I'll try to get to looking at your PR before the end of the week.

@vincetran vincetran added Push Notifications Related to the push notifications category bug Something isn't working and removed pending-triage Issue is pending triage labels Jun 20, 2024
@jamilsaadeh97
Copy link
Contributor Author

Hi @vincetran looking forward for your feedback!

@tylerjroach tylerjroach added feature-request Request a new feature and removed bug Something isn't working labels Jun 21, 2024
Copy link
Contributor

github-actions bot commented Jul 5, 2024

This issue is now closed. Comments on closed issues are hard for our team to see.
If you need more assistance, please open a new issue that references this one.

@mattcreaser mattcreaser added the pending-release Code has been merged but pending release label Jul 8, 2024
@mattcreaser mattcreaser reopened this Jul 8, 2024
@mattcreaser
Copy link
Member

Did not mean to auto-close this. We typically only close issues once the fix is released. This PR has been merged and will be included in our next release.

@mattcreaser
Copy link
Member

This fix we released in Amplify 2.20.0 😄

@github-actions github-actions bot removed the pending-release Code has been merged but pending release label Jul 15, 2024
Copy link
Contributor

This issue is now closed. Comments on closed issues are hard for our team to see.
If you need more assistance, please open a new issue that references this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Request a new feature Push Notifications Related to the push notifications category
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants