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: currency in coa import #34174

Merged
merged 2 commits into from
Feb 27, 2023
Merged

fix: currency in coa import #34174

merged 2 commits into from
Feb 27, 2023

Conversation

Vishnu7025
Copy link
Contributor

Added currency column in Chart of Accounts Importer

@github-actions github-actions bot added the needs-tests This PR needs automated unit-tests. label Feb 22, 2023
@codecov
Copy link

codecov bot commented Feb 24, 2023

Codecov Report

Merging #34174 (e3c000d) into develop (a06d240) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #34174      +/-   ##
===========================================
- Coverage    64.05%   64.05%   -0.01%     
===========================================
  Files          810      810              
  Lines        59307    59309       +2     
===========================================
  Hits         37992    37992              
- Misses       21315    21317       +2     
Impacted Files Coverage Δ
...ype/account/chart_of_accounts/chart_of_accounts.py 76.87% <ø> (ø)
...of_accounts_importer/chart_of_accounts_importer.py 0.00% <0.00%> (ø)

@deepeshgarg007 deepeshgarg007 added the backport version-14-hotfix backport to version 14 label Feb 27, 2023
@deepeshgarg007 deepeshgarg007 merged commit 81651be into frappe:develop Feb 27, 2023
deepeshgarg007 pushed a commit that referenced this pull request Feb 27, 2023
* fix: currency in coa import

(cherry picked from commit 19c0b7a)

* chore: change column label

(cherry picked from commit e3c000d)

---------

Co-authored-by: vishnu <vishnuviswambaran2002@gmail.com>
frappe-pr-bot pushed a commit that referenced this pull request Feb 28, 2023
## [14.17.2](v14.17.1...v14.17.2) (2023-02-28)

### Bug Fixes

* conversion factor not set ([089c7d0](089c7d0))
* currency in coa import ([#34174](#34174)) ([4d92d46](4d92d46))
* default date in Subcontracting reports ([5fce8e2](5fce8e2))
* german translations ([#31732](#31732)) ([88a781f](88a781f))
* incorrect acc depr amount if multiple FBs with straight line or manual method ([dda6bae](dda6bae))
* incorrect color in the BOM Stock Report ([001ed9e](001ed9e))
* manual depr schedule ([971c072](971c072))
* multiple pos conversion issue resolved ([db964e8](db964e8))
* not able to repost gl entries ([ae0318e](ae0318e))
* permission error while calling get_work_order_items ([3ea90ee](3ea90ee))
* pos return throwing amount greater than grand total ([9cd7b27](9cd7b27))
* Remove missing DocField in fetch_from ([dc6ae46](dc6ae46))
* set `from_warehouse` and `to_warehouse` while mapping SE ([80e23d0](80e23d0))
* **test:** use standalone method to fetch work orders from SO ([1719884](1719884))
* ui freeze on item selection in sales invoice ([1750ed4](1750ed4))
* user shouldn't able to make item price for item template ([fb8e45d](fb8e45d))
* zero division error while making LCV ([1859be6](1859be6))

### Performance Improvements

* fetch SLE's on demand and memoize ([db1f17e](db1f17e))

### Reverts

* Revert "fix: Concurrency issues in Sales and Purchase returns" (#34202) ([9341d3e](9341d3e)), closes [#34202](#34202) [#34019](#34019)
@barredterra barredterra mentioned this pull request Mar 7, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 26, 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 needs-tests This PR needs automated unit-tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants