-
Notifications
You must be signed in to change notification settings - Fork 7.1k
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: add the company in payment request bcz delete company transactions #42664
fix: add the company in payment request bcz delete company transactions #42664
Conversation
I think the company field is implicitly set. On a multi-company setup, this might incorrectly set the same company for all PR's. Can you test this? |
Hi @ruthra-kumar, Video is long but check it out. multi-company.with.PR.mp4 |
This is exactly what I had suspected. Process flow should not rely upon user setting the correct session defaults. Not every user will do that. pr_company_after.mp4Explicit is always better than Implicit. |
With e300884, above issue should be fixed. |
…ns (backport #42664) (#42982) * fix: add the company in payment request bcz delete company transactions (cherry picked from commit 12834cc) # Conflicts: # erpnext/accounts/doctype/payment_request/payment_request.json * fix: link company when make payment request (cherry picked from commit e300884) * fix: add the company in payment request bcz delete company transactions --conflicts --------- Co-authored-by: Nihantra Patel <nihantra@frappe.io> Co-authored-by: Nihantra C. Patel <141945075+Nihantra-Patel@users.noreply.github.com>
## [15.34.1](v15.34.0...v15.34.1) (2024-09-05) ### Bug Fixes * add the company in payment request bcz delete company transactions (backport [#42664](#42664)) ([#42982](#42982)) ([42e7725](42e7725)) * added app permission check for apps page ([a35ce12](a35ce12)) * adjust price insertion logic for internal suppliers/customers ([#42988](#42988)) ([daa75ee](daa75ee)) * auto reorder material request mail issue (backport [#43066](#43066)) ([#43068](#43068)) ([d2b2002](d2b2002)) * **capitalization:** debit cwip account instead of fixed asset account ([#42857](#42857)) ([f3c60ea](f3c60ea)) * company accounts setup_queries ([b99cdb5](b99cdb5)) * disabled batches showing in the list (backport [#43024](#43024)) ([#43069](#43069)) ([56dad7d](56dad7d)) * don't allow capitalizing only service item for new composite asset ([a833010](a833010)) * improve asset item matching logic ([3bb1867](3bb1867)) * indentation ([4d7c0c0](4d7c0c0)) * link Purchase Invoice and Receipt Items to Asset ([1121c66](1121c66)) * retain date filter when redirecting in Profit and Loss report ([f0e3fb4](f0e3fb4)) * typeerror on Payment Entry ([6d51d14](6d51d14)) * typerror on default_currency ([7d6984c](7d6984c)) * update develop_version in hooks ([6c8e0fd](6c8e0fd)) * validate component quantity according to BOM (backport [#43011](#43011)) ([#43014](#43014)) ([fee2255](fee2255))
version 15
fixes: #42659
Before:
before_payment_request.mp4
After:
after_payment_request.mp4