Fixed issue with empty state field in DTO

[jcadam14]

Closes #236

Fixed an issue with empty state codes failing on sending an associated DTO back.

[github-actions]

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
src/regtech_user_fi_management/entities/models
dto.py
Project Total

This report was generated by python-coverage-comment-action

[hkeeler]

Looks good, though black seems to be grumpy with some part of this change.

On a side note, I would expect that to prevent me from merging this, but it's not. Seems like all the branch protection rules are correct too. 🤔

[jcadam14]

Looks good, though black seems to be grumpy with some part of this change.

On a side note, I would expect that to prevent me from merging this, but it's not. Seems like all the branch protection rules are correct too. 🤔

Yeah I was trying to fix black but was having git issues, all resolved

[jcadam14]

Looks good, though black seems to be grumpy with some part of this change.

On a side note, I would expect that to prevent me from merging this, but it's not. Seems like all the branch protection rules are correct too. 🤔

Also yeah I've noticed that you can approve, and merge, even if a security or linting test fails. Actually good with the security thing because Snyk is continually failing for no reason on one of our repos (api-commons I believe) and it would keep us from ever merging anything since we have no control of that thing.