Fix repeated top post notifications minds#3200

Ticket(s) / Related Merge Requests

minds#3200 (closed)

Summary of Changes

Fixes repeated top post notifications by limiting the query only to posts in the past 24 hours

Testing Considerations

You will need to use the CLI to check different lists:

First, get your channel GUID from https://fix-apns-no-title-m3202.minds.io//api/v1/channel/{{username}}

1. Log in to your account
2. Enabled push notifications (from settings or notifications dropdown)
3. Post an image/video without any title or description
4. close the tab

here it gets a bit technical, get into the php-fpm container of the fix-apns-no-title-m3202 pod and run:

cd engine; source /vault/secrets/env; php cli.php Notification sendTopPostPush --user_guid={{YOUR GUID}} --target_list=AllDevices

you should now receive a notification. Make sure the notification looks alright.

Deployment Considerations

TBD

Regression Scope

top post notifications

Platform Affected (web, mobile, etc)

N/A

Developer Testing Completed

TBD

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