Skip to content

#1284 fix inbox pagination after loading all messages#1287

Merged
tomholub merged 1 commit intomasterfrom
bugfix/issue-1284-inbox-pagination
Jan 11, 2022
Merged

#1284 fix inbox pagination after loading all messages#1287
tomholub merged 1 commit intomasterfrom
bugfix/issue-1284-inbox-pagination

Conversation

@sosnovsky
Copy link
Collaborator

@sosnovsky sosnovsky commented Jan 11, 2022

This PR fixes inbox pagination after loading all messages/threads.

close #1284


Tests (delete all except exactly one):

  • Difficult to test (explain why) - there is no easy way to calculate how many messages were loaded in appium, can be done in separate task if such test is needed.

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities
  • is documented clearly and usefully, or doesn't need documentation

@sosnovsky sosnovsky requested a review from tomholub January 11, 2022 12:30
@tomholub tomholub enabled auto-merge (squash) January 11, 2022 12:32
@tomholub
Copy link
Collaborator

@fcvakintos please test

@tomholub tomholub merged commit aa37ac3 into master Jan 11, 2022
@tomholub tomholub deleted the bugfix/issue-1284-inbox-pagination branch January 11, 2022 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

BUG: Inbox screen displays only 20 emails after updating inbox screen on e2e account

2 participants