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: project filtering based on company in P&L Report #35943

Merged

Conversation

GursheenK
Copy link
Member

Added company specific Project Filtering in P&L report.

Previously the P&L Report filter for Project showed projects for all companies instead of for the selected company.


Project belonging to company - Twiga
Screenshot 2023-06-30 at 12 15 48 PM


Project filter shows the project even though another company is selected.
Screenshot 2023-06-30 at 12 16 10 PM


Project filter now shows only projects for selected company.
Screenshot 2023-06-30 at 12 20 13 PM

@codecov
Copy link

codecov bot commented Jun 30, 2023

Codecov Report

Merging #35943 (84d4888) into develop (1d1103f) will increase coverage by 0.00%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #35943     +/-   ##
==========================================
  Coverage    64.15%   64.15%             
==========================================
  Files          813      785     -28     
  Lines        62217    60485   -1732     
==========================================
- Hits         39914    38803   -1111     
+ Misses       22303    21682    -621     

see 45 files with indirect coverage changes

@deepeshgarg007 deepeshgarg007 added the backport version-14-hotfix backport to version 14 label Jun 30, 2023
@deepeshgarg007 deepeshgarg007 merged commit be5cb1a into frappe:develop Jun 30, 2023
deepeshgarg007 pushed a commit that referenced this pull request Jul 1, 2023
* fix: project filtering in P&L Report

(cherry picked from commit 904ca74)

* fix: show projects with no company value set

(cherry picked from commit ce252a0)

* fix: make company field mandatory in project doctype

(cherry picked from commit 84d4888)

---------

Co-authored-by: Gursheen Anand <gursheen@frappe.io>
frappe-pr-bot pushed a commit that referenced this pull request Jul 5, 2023
# [14.29.0](v14.28.1...v14.29.0) (2023-07-05)

### Bug Fixes

* Expense Account filter in Sales Invoice ([#35944](#35944)) ([a485e4e](a485e4e))
* get base grand total while pulling reference details in PE ([2e2c23a](2e2c23a))
* handle loan_repayment's posting_date datetime in bank_clearance_summary report ([#36004](#36004)) ([937e1fb](937e1fb))
* Netherlands - Grootboekschema COA structure ([#35991](#35991)) ([13f3ebf](13f3ebf))
* project filtering based on company in P&L Report ([#35943](#35943)) ([8de1d86](8de1d86))
* remove debug flag from sql ([646440f](646440f))
* reposting has not changed valuation rate ([c0c693d](c0c693d))
* Update no copy for received_qty field ([#35965](#35965)) ([c330f47](c330f47))

### Features

* **accounts:** standardize additional columns implementation for sales/purchase reports ([#36000](#36000)) ([47c6d90](47c6d90))
* add method for ordered quantity in supplier scorecard (backport [#35930](#35930)) ([#35968](#35968)) ([a974091](a974091))
* add voucher-wise balance report logic ([6842902](6842902))
* allow the partial return of components against SCO (backport [#35935](#35935)) ([#35938](#35938)) ([6f50ad6](6f50ad6))
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport version-14-hotfix backport to version 14
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants