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

Account Summary in the Transaction List Block Changed #5752

Closed
2 tasks done
FrankGrand opened this issue Feb 14, 2024 · 0 comments
Closed
2 tasks done

Account Summary in the Transaction List Block Changed #5752

FrankGrand opened this issue Feb 14, 2024 · 0 comments
Labels
Fixed in v17.0 Status: Confirmed It's clear what the subject of the issue is about, and what the resolution should be. Type: Bug Confirmed bugs or reports that are very likely to be bugs.

Comments

@FrankGrand
Copy link

Description

After our deploy to prealpha 1.17.0.11 our finance team noticed that the account summary at the bottom of the transaction page changed behaviors.

Actual Behavior

It now only totals whatever the transactions are on the page you are viewing instead of the totals for the entire query. For example, if you search for all transactions in Feb and have the page limited to 50 results, you'll only get the totals for the first 50 transactions instead of the totals for all results (across multiple pages).

Expected Behavior

We expect the previous behavior that all transactions in the query are totaled.

Steps to Reproduce

  • Go to Transaction page (on or post 1.17.0.11)
  • Scroll down to 'Total Results'
  • See error

Issue Confirmation

  • Perform a search on the Github Issues to see if your bug or enhancement is already reported.
  • Try to reproduce the problem on a fresh install or on the demo site.

Rock Version

1.17.0.11

Client Culture Setting

en-US

@sparkdevnetwork-service sparkdevnetwork-service added Status: Confirmed It's clear what the subject of the issue is about, and what the resolution should be. Type: Bug Confirmed bugs or reports that are very likely to be bugs. and removed Status: Confirmed It's clear what the subject of the issue is about, and what the resolution should be. labels Feb 14, 2024
@chead4 chead4 added the Status: Confirmed It's clear what the subject of the issue is about, and what the resolution should be. label Feb 14, 2024
joshuahenninger added a commit that referenced this issue Feb 20, 2024
…wed total amounts for the current page. (Fixes #5752)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fixed in v17.0 Status: Confirmed It's clear what the subject of the issue is about, and what the resolution should be. Type: Bug Confirmed bugs or reports that are very likely to be bugs.
Projects
None yet
Development

No branches or pull requests

4 participants