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: map Accounting Dimensions for Bank Entry against Payroll Entry #29142

Merged

Conversation

ruchamahabal
Copy link
Member

@ruchamahabal ruchamahabal commented Jan 4, 2022

Steps to Replicate:

  1. Create an Accounting Dimension eg: Branch
    image

  2. Create a Payroll Entry, submit salary slips.

  3. Make Bank Entry against Payroll Entry. This will create a Journal Entry to book salary payment.

  4. This Journal Entry won't get submitted due to missing mandatory accounting dimension.

    Kapture 2022-01-04 at 19 29 34

Fix:

Accounting Dimensions get mapped while making accrual JV on salary slip submission but not when making Bank Entry. Map accounting dimensions in this method too.

Kapture 2022-01-04 at 19 32 05

@ruchamahabal
Copy link
Member Author

@Mergifyio backport version-13-hotfix version-13-pre-release

@mergify
Copy link
Contributor

mergify bot commented Jan 4, 2022

backport version-13-hotfix version-13-pre-release

🟠 Waiting for conditions to match

  • merged [:pushpin: backport requirement]

@ruchamahabal
Copy link
Member Author

Unrelated test failures

@deepeshgarg007
Copy link
Member

Failing test cases are unrelated, merging this PR

@mergify
Copy link
Contributor

mergify bot commented Jan 4, 2022

backport version-13-hotfix version-13-pre-release

✅ Backports have been created

deepeshgarg007 added a commit that referenced this pull request Jan 4, 2022
…-29142

fix: map Accounting Dimensions for Bank Entry against Payroll Entry (backport #29142)
deepeshgarg007 added a commit that referenced this pull request Jan 4, 2022
…se/pr-29142

fix: map Accounting Dimensions for Bank Entry against Payroll Entry (backport #29142)
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.

2 participants