Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BUG - Sev-3 - All - Secure Messaging: Inbox displaying as empty on pages that are past 100 messages #10379

Open
5 tasks
rbontrager opened this issue Dec 12, 2024 · 1 comment
Labels
bug Used to identify a bug ticket that will be worked through the bug process Health Tickets are tied to the Health Product Team Pms scrubbed Added to newly-written bug tickets after QA has reviewed them for clarity and completeness sev-3 Lowest bug severity level based on QA bug severity scale - QA to assign this

Comments

@rbontrager
Copy link
Contributor

What happened?

The inbox is coming in as empty when you use pagination to go to a page past 100 messages.

Specs:

  • Device:
  • OS:
  • User Account:
  • Accessibility State:

Steps to Reproduce

  1. Login to user +226 (or any user with over 100 messages)
  2. Using the next page feature go to the inbox for messages 101-110. Notice that the inbox is coming back as empty when it isn't.

Desired behavior

Acceptance Criteria

Bug Severity - BE SURE TO ADD THE SEVERITY LABEL

See Bug Tracking for details on severity levels

  • Impact: High Low
  • Frequency: High Low
  • 3 - Low
  • 2 - High
  • 1 - Critical

Linked to Story

Screen shot(s) and additional information

Full JSON response for services related to issue (expand/collapse)

Ticket Checklist

  • Steps to reproduce are defined
  • Desired behavior is added
  • Labels added (front-end, back-end, global, Health, relevant feature, accessibility, etc)
  • Estimate of 1 added (for front-end tickets)
  • Added either to the relevant feature epic (if found during new feature implementation), or the relevant team's bug epic (Global, Health & Benefits, API, QART)
@rbontrager rbontrager added bug Used to identify a bug ticket that will be worked through the bug process Health Tickets are tied to the Health Product Team Pms scrubbed Added to newly-written bug tickets after QA has reviewed them for clarity and completeness sev-3 Lowest bug severity level based on QA bug severity scale - QA to assign this labels Dec 12, 2024
@alexandec alexandec self-assigned this Dec 12, 2024
@alexandec
Copy link
Contributor

alexandec commented Dec 12, 2024

The issue is that we are attempting to fetch 5000 messages:

GET /mobile/v0/messaging/health/folders/0/messages?page=1&per_page=5000&useCache=false

However, we are actually limited to 100 messages based on how the endpoint is implemented. 100 is the max we can receive with a single call.

Looks like the issue started in May with the React Query migration of Secure Messaging endpoints. What we used to do before that was:

return get<SecureMessagingFolderMessagesGetData>(`/v0/messaging/health/folders/${folderID}/messages`, {
  page: page.toString(),
  per_page: DEFAULT_PAGE_SIZE.toString(),
} as Params)

so we only fetched messages 10 at a time, and fetched 10 more as the user moved to the next page. We could potentially return to that form of user-initiated fetching in order to resolve the issue.

Additional notes:

  • Demo mode, and possibly unit/E2E tests, would need to be updated accordingly.
  • The same issue probably occurs with other folders, not just the inbox.

Slack thread: https://adhoc.slack.com/archives/C067S0FRP0C/p1734027121247159

@alexandec alexandec removed their assignment Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Used to identify a bug ticket that will be worked through the bug process Health Tickets are tied to the Health Product Team Pms scrubbed Added to newly-written bug tickets after QA has reviewed them for clarity and completeness sev-3 Lowest bug severity level based on QA bug severity scale - QA to assign this
Projects
None yet
Development

No branches or pull requests

2 participants