Skip to content

Feat top post undescriptive minds#3295

Manish requested to merge feat/top-post-undescriptive-m3295 into master

Ticket(s) / Related Merge Requests

minds#3295 (closed)

Summary of Changes

This MR uses the name of the owner instead of the username in notifications and supports links as top post notifications by using the title of the rich embed as notification body and changing the title to "Jack posted a link"

Testing Considerations

Follow the testing instructions here to get top post notifications and make sure the notifications appear according to the acceptance criteria

Deployment Considerations

N/A

Regression Scope

all top posts

Platform Affected (web, mobile, etc)

notifcations

Developer Testing Completed

local

Screenshots / Screen Recording

Screen_Shot_2022-07-27_at_13.41.21

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