bug/10330-emptyPageAfterMovingMessage

[alexandec]

Description of Change

If a user is on page 2 or greater in a Secure Messaging folder, the page has only one message, and they move that message to a different folder, go back to the previous page. This avoids showing them an empty message list.

The change applies to the inbox as well as all other folders.

Screenshots/Video

Before:

move-message-before.mp4

After:

move-message-after.mp4

Testing

• Tested on iOS
• Tested on Android

Reviewer Validations

User is brought back into the folder, on the 'prior' page (screen showing the 1-10 messages if you just deleted the 11th, the 11-20 messages if you just deleted the 21st, etc)

PR Checklist

Reviewer: Confirm the items below as you review

• PR is connected to issue(s)
• Tests are included to cover this change (when possible)
• No magic strings (All string unions follow the Union -> Constant type pattern)
• No secrets or API keys are checked in
• All imports are absolute (no relative imports)
• New functions and Redux work have proper TSDoc annotations

For QA

Run a build for this branch

[dumathane]

I'm a little concerned about what happens when you have a folder with only one message and move it? Did you try that test case Chris?

[alexandec]

@dumathane the conditional here checks that the current page is > 1 before setting the page to page - 1. So the code won't run if there's only one page of items. I can still test that on staging to be sure though

[alexandec]

@dumathane works fine, you just get the "You don't have any messages in this folder" message. Video:

one.message.mp4