Skip to content

Only Clear Bank Journal if in absolute violation of company.#21

Open
richard-willdooit wants to merge 2 commits intoForgeFlow:10.0from
richard-willdooit:10.0-account-bank-statement-journal-default
Open

Only Clear Bank Journal if in absolute violation of company.#21
richard-willdooit wants to merge 2 commits intoForgeFlow:10.0from
richard-willdooit:10.0-account-bank-statement-journal-default

Conversation

@richard-willdooit
Copy link

When the default comes up, if it calls onchange (depending what was
there) - which clears the default journal, and the journal type, meaning
the journal is not even selectable any more by domain.

When the default comes up, if it calls onchange (depending what was
there) - which clears the default journal, and the journal type, meaning
the journal is not even selectable any more by domain.
@codecov-io
Copy link

codecov-io commented Feb 6, 2018

Codecov Report

Merging #21 into 10.0 will decrease coverage by 0.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##             10.0      #21      +/-   ##
==========================================
- Coverage   52.58%   52.57%   -0.02%     
==========================================
  Files         118      118              
  Lines        4931     4932       +1     
==========================================
  Hits         2593     2593              
- Misses       2338     2339       +1
Impacted Files Coverage Δ
mcfix_account/models/account_bank_statement.py 20.95% <0%> (-0.21%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f787c95...66eec6f. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants