Skip to content

If no child_ids on company, use empty tuple.#19

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

If no child_ids on company, use empty tuple.#19
richard-willdooit wants to merge 2 commits intoForgeFlow:10.0from
richard-willdooit:10.0-account-bank-statement-no-childids

Conversation

@richard-willdooit
Copy link

No description provided.

@JordiBForgeFlow
Copy link
Contributor

Thank you @richard-willdooit. We are continuing our project in 11.0. So we will consider your fixes on that branch. cc @mreficent

Also avoids null parameter being passed to sql
@codecov-io
Copy link

Codecov Report

Merging #19 into 10.0 will decrease coverage by 0.02%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##             10.0      #19      +/-   ##
==========================================
- Coverage   52.58%   52.56%   -0.03%     
==========================================
  Files         118      118              
  Lines        4931     4933       +2     
==========================================
  Hits         2593     2593              
- Misses       2338     2340       +2
Impacted Files Coverage Δ
mcfix_account/models/account_bank_statement.py 20.75% <0%> (-0.4%) ⬇️

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...e03e90e. 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.

3 participants