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: removed "fetch_from" #36365

Merged
merged 1 commit into from
Jul 28, 2023
Merged

fix: removed "fetch_from" #36365

merged 1 commit into from
Jul 28, 2023

Conversation

AlexandreLumertz
Copy link
Contributor

*fix: removed ("fetch_from": "goal.objective")

  • The field was disabled due to "fetch_from".
  • And it doesn't carry a value, perhaps because the "goal" field belongs to the parent doctype.

@codecov
Copy link

codecov bot commented Jul 27, 2023

Codecov Report

Merging #36365 (9a4fa7e) into develop (dac9fd6) will increase coverage by 0.00%.
Report is 8 commits behind head on develop.
The diff coverage is n/a.

❗ Current head 9a4fa7e differs from pull request most recent head 1c687a4. Consider uploading reports for the commit 1c687a4 to get more accurate results

Additional details and impacted files
@@           Coverage Diff            @@
##           develop   #36365   +/-   ##
========================================
  Coverage    64.25%   64.25%           
========================================
  Files          785      785           
  Lines        60965    60965           
========================================
+ Hits         39173    39174    +1     
+ Misses       21792    21791    -1     

see 1 file with indirect coverage changes

* fix: removed ("fetch_from": "goal.objective")
The field ended up being disabled because of this.
@s-aga-r s-aga-r merged commit bc6cbb9 into frappe:develop Jul 28, 2023
10 checks passed
s-aga-r pushed a commit that referenced this pull request Jul 28, 2023
fix: removed "fetch_from"

* fix: removed ("fetch_from": "goal.objective")
The field ended up being disabled because of this.

(cherry picked from commit 1c687a4)

Co-authored-by: xdlumertz <alexandrelumertz@gmail.com>
s-aga-r pushed a commit that referenced this pull request Jul 28, 2023
fix: removed "fetch_from"

* fix: removed ("fetch_from": "goal.objective")
The field ended up being disabled because of this.

(cherry picked from commit 1c687a4)

Co-authored-by: xdlumertz <alexandrelumertz@gmail.com>
frappe-pr-bot pushed a commit that referenced this pull request Aug 1, 2023
## [13.52.8](v13.52.7...v13.52.8) (2023-08-01)

### Bug Fixes

* added validation for unique serial numbers in pos invoice ([#36302](#36302)) ([a165b37](a165b37))
* only publish repost progress to doc subscriber (backport [#36400](#36400)) ([#36403](#36403)) ([e9df064](e9df064))
* removed "fetch_from" (backport [#36365](#36365)) ([#36387](#36387)) ([c574494](c574494))

### Performance Improvements

* use `LEFT JOIN` instead of `NOT EXISTS` (backport [#36221](#36221)) ([#36384](#36384)) ([cdc86bd](cdc86bd))
frappe-pr-bot pushed a commit that referenced this pull request Aug 1, 2023
# [14.33.0](v14.32.1...v14.33.0) (2023-08-01)

### Bug Fixes

* allow fully depreciated existing assets ([#36378](#36378)) ([43b85c5](43b85c5))
* change fieldtype from Currency to Float for the valuation rate in the stock report ([93bd4c7](93bd4c7))
* cost center filter for fetching payments ([c9daa85](c9daa85))
* Defined "Open" Status as default (backport [#36421](#36421)) ([#36424](#36424)) ([33a9477](33a9477))
* filter by cost center in trial balance ([02428b4](02428b4))
* german translations ([9c8e2a3](9c8e2a3))
* GL Entries should not be splitted based on cost center allocation in PCV ([ade13e6](ade13e6))
* group item reorder by (warehouse, material_request_type) (backport [#35818](#35818)) ([#36425](#36425)) ([516191b](516191b))
* Ignore account closing balance for financial statement ([800417e](800417e))
* in payment_entry 'Unallocated Amount' cal is broken ([#36369](#36369)) ([cd3b85c](cd3b85c))
* incorrect `idx` on JE's after reconciliation ([80eb875](80eb875))
* incorrect qty set in the serial no picker ([57b19a5](57b19a5))
* incorrect usage `get_cached_value` on single doctypes ([8f72a68](8f72a68))
* **Item Group:** allow root deletion ([1a6be5e](1a6be5e))
* job card suggest holiday as start date (backport [#35958](#35958)) ([#36423](#36423)) ([29ddd26](29ddd26))
* Job Card validation fixed when displaying total completed quantity ([7b3bcd3](7b3bcd3))
* multiple issues related to Production Plan (backport [#36377](#36377)) ([#36381](#36381)) ([a799e1b](a799e1b))
* not able to make material request (backport [#36416](#36416)) ([#36426](#36426)) ([99e7406](99e7406))
* only publish repost progress to doc subscriber (backport [#36400](#36400)) ([#36402](#36402)) ([32bdb7c](32bdb7c))
* overallocation validation misfire on normal invoices ([#36349](#36349)) ([09af485](09af485))
* paid_amount when the group is mode of payment ([50ef358](50ef358))
* process_owner is not link User (backport [#36420](#36420)) ([#36422](#36422)) ([289dc36](289dc36))
* removed "fetch_from" (backport [#36365](#36365)) ([#36386](#36386)) ([6d051f5](6d051f5))
* root type in account map for balance sheet ([#36303](#36303)) ([5b56296](5b56296))
* show invoices name instead of object address ([e802f0c](e802f0c))
* show tds & tcs separately ([619b0fe](619b0fe))
* test with None conditon in PE ([479cab0](479cab0))
* typo in loyalty program throw message ([#36432](#36432)) ([782a4d1](782a4d1))

### Features

* add party type filter ([32fea64](32fea64))

### Performance Improvements

* avoid full table scan in sle count check ([#36428](#36428)) ([04f9915](04f9915))
* move project status reminder to hourly (backport [#36372](#36372)) ([#36373](#36373)) ([4ac60cd](4ac60cd))
* use `LEFT JOIN` instead of `NOT EXISTS` (backport [#36221](#36221)) ([#36383](#36383)) ([26a0b3b](26a0b3b))
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants