Skip to content

1x remind only minds#4340

Olivia Madrid requested to merge feat/one-remind-m4340 into master

Ticket(s) / Related Merge Requests

minds#4340 (closed)

Summary of Changes

Two new endpoints added for handling requirements of only allowing 1x remind of a post at a time:

  • check if the user has reminded this post before
    • endpoint: GET api/v3/newsfeed/activity/has-reminded/{guidOfOriginalPost})
    • response: includes has-reminded property that is either true or false
  • delete all reminds that the user has made of this post
    • endpoint: DELETE api/v3/newsfeed/activity/remind/{guidOfOriginalPost}
    • response: is just success (true or false)

If you are able to bypass the frontend restrictions and get to the endpoint where you are trying to make a new remind when you already have made one, an error is thrown.

Testing Considerations

Creating and deleting reminds

Deployment Considerations

Inform mobile on deployment

Regression Scope

Creating/undoing reminds.

Platform Affected (web, mobile, etc)

Both. But since the changes are only accessible through new endpoints, it should not break anything for mobile.

Developer Testing Completed

  • Whether the user has reminded is accurate
  • Reminds of that post are deleted for that user only

Screenshots / Screen Recording

See front MR front!2444 (merged)

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
Edited by Olivia Madrid

Merge request reports

Loading