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

fix(general ledger): link to journal rather than account stmt #7792

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jniles
Copy link
Collaborator

@jniles jniles commented Oct 19, 2024

This commit fixes a long broken link in the General Ledger that links to the account statement report. This report was removed some time ago. It replaces this link with the filtered posting journal, which emulates the same behavior. It allows users to have a dynamic look at the transactions affecting the account in question.

Closes #7787.

To test this PR, it's best to use a database with posted records, such as one of the production databases.

  1. Navigate to Finance > General Ledger
  2. Find a row that has balances for any of the months of the year (not just opening balances)
  3. Click the "View in Posting Journal" action in the dropdown.

You should arrive at the posting journal, filtered on that year and account, with both posted and unposted transaction.

This commit fixes a long broken link in the General Ledger that links to
the account statement report. This report was removed some time ago.  It
replaces this link with the filtered posting journal, which emulates the
same behavior.  It allows users to have a dynamic look at the
transactions affecting the account in question.

Closes IMA-WorldHealth#7787.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make the "View Account Statement" link in the General Ledger link to a filtered journal
2 participants