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: "is_query_report" check for Vehicle Expenses Report #1383

Merged
merged 2 commits into from
Feb 5, 2024

Conversation

Jeevansyriac
Copy link
Contributor

image

When we click on 'Vehicle Expenses' in the workspace, it does not navigate to the report view; instead, it navigates to the doctype.

image

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (eded9ce) 72.46% compared to head (c8e4c82) 72.51%.
Report is 2 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1383      +/-   ##
===========================================
+ Coverage    72.46%   72.51%   +0.04%     
===========================================
  Files          201      201              
  Lines        10986    10986              
===========================================
+ Hits          7961     7966       +5     
+ Misses        3025     3020       -5     

see 2 files with indirect coverage changes

@ruchamahabal ruchamahabal merged commit 6802174 into frappe:develop Feb 5, 2024
6 checks passed
mergify bot pushed a commit that referenced this pull request Feb 5, 2024
* fix: "is_query_report" check for Vehicle Expenses Report

* fix: timestamp updated

(cherry picked from commit 6802174)
mergify bot pushed a commit that referenced this pull request Feb 5, 2024
* fix: "is_query_report" check for Vehicle Expenses Report

* fix: timestamp updated

(cherry picked from commit 6802174)
ruchamahabal pushed a commit that referenced this pull request Feb 5, 2024
) (#1387)

* fix: "is_query_report" check for Vehicle Expenses Report

* fix: timestamp updated

(cherry picked from commit 6802174)

Co-authored-by: Jeevan syriac <[email protected]>
ruchamahabal pushed a commit that referenced this pull request Feb 5, 2024
) (#1388)

* fix: "is_query_report" check for Vehicle Expenses Report

* fix: timestamp updated

(cherry picked from commit 6802174)

Co-authored-by: Jeevan syriac <[email protected]>
frappe-pr-bot pushed a commit that referenced this pull request Feb 6, 2024
## [14.21.5](v14.21.4...v14.21.5) (2024-02-06)

### Bug Fixes

* "is_query_report" check for Vehicle Expenses Report (backport [#1383](#1383)) ([#1387](#1387)) ([26b9404](26b9404))
* employee advance option added to payment entry (backport [#1380](#1380)) ([#1385](#1385)) ([db530c1](db530c1))
* **Leave Type:** translatability of tour step (backport [#1371](#1371)) ([#1372](#1372)) ([f2d9039](f2d9039))
frappe-pr-bot pushed a commit that referenced this pull request Feb 6, 2024
# [15.11.0](v15.10.0...v15.11.0) (2024-02-06)

### Bug Fixes

* "is_query_report" check for Vehicle Expenses Report (backport [#1383](#1383)) ([#1388](#1388)) ([bee2c3c](bee2c3c))
* accounts validation ([5d1d030](5d1d030))
* employee advance option added to payment entry ([#1380](#1380)) ([b51df74](b51df74))
* **Leave Type:** translatability of tour step (backport [#1371](#1371)) ([#1373](#1373)) ([3fa14df](3fa14df))
* reset old code field values on update for better readability in the doctype form ([79f4616](79f4616))
* sanitize condition and formulae expressions before validation ([f17608f](f17608f))
* sanitize structure fields + reset fields on `on_update` for readability ([f626c4e](f626c4e))
* save formula/condition changes in version ([26f6b75](26f6b75))
* separate tab for HR and Payroll in company ([b9018d9](b9018d9))
* **Shift Attendance:** conversion of datetime to time for same date for None values (backport [#1345](#1345)) ([#1391](#1391)) ([d202c16](d202c16))
* update only submitted salary structures ([3870a0e](3870a0e))

### Features

* add autocompletion to code fields ([3fed929](3fed929))
* add autocompletions for Salary Structure and Salary Slip fields ([4956792](4956792))
* add employee field autocompletions ([f86b484](f86b484))
* add option to sync formula and condition for existing structures ([076f23f](076f23f))
* add Salary Structure Assignment to autocompletions ([d0dca02](d0dca02))
* add success toast ([cd8ac08](cd8ac08))
* add syntax highlighting to code fields ([160d9cb](160d9cb))
* add warning for unset accounts on insert ([8c55170](8c55170))
* increase base/variable weight in autocompletions ([7199511](7199511))
* **Salary Structure:** add syntax highlighting ([37641a6](37641a6))
* setup autocompletions for salary structure fields ([bd2d3de](bd2d3de))
* show alert in case of no Salary Structures to be updated ([7a54e4e](7a54e4e))
* show Salary Structures to be updated in confirm dialog ([bf96267](bf96267))
* use form links for structures in confirm dialog ([907eb93](907eb93))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants