Skip to content

Improve push notifs for post subscription notifs minds#4601

Olivia Madrid requested to merge feat/improve-post-push-m4601 into master

Ticket(s) / Related Merge Requests

minds#4601 (closed)

Summary of Changes

For post subscription notifications,

  • the title will be the user name
  • the body will be populated using the following logic:
    • Does it have a message? If yes, return the message (this covers text posts, rich-embeds with messages)
    • Does it have a title? Return the title. (this covers message-less rich-embeds and blogs as well as titled media posts)
    • Does it have a video media attachment? Return 'Posted a video'.
    • Does it have one or more images attached? If one image, return 'Posted an image'. If multiple images, return 'Posted images'.

Testing Considerations

Looking at the PushNotificationSpec tests

Deployment Considerations

Regression Scope

Post subscription push notifications

Platform Affected (web, mobile, etc)

Both

Developer Testing Completed

Spec tests created

Screenshots / Screen Recording

Does this impact

  • Localization
  • Dark/light mode
  • Guest mode

Definition of Done Checklist

  • The Acceptance Criteria has been met
  • Code is tested: Testing includes unit/spec, E2E/automated and manual testing
  • Merge requests description has been filled out

Merge request reports