Closed adamcharnock closed 3 months ago
Attention: Patch coverage is 85.63830%
with 54 lines
in your changes missing coverage. Please review.
Project coverage is 92.05%. Comparing base (
81b90df
) to head (ef60492
). Report is 15 commits behind head on master.:exclamation: Current head ef60492 differs from pull request most recent head eb161a2
Please upload reports for the commit eb161a2 to get more accurate results.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
My apologies for all this being in one PR, I know it isn't ideal. It is mostly fairly straightforward, except that a lot of lines were touched by the removal of
django-model-utils
. I'm going to get this merged now and then do the remaining changes hopefully in their own PRs.Changelog
DEFAULT_CURRENCY
rather than all available currencies (trading accounts should be the only accounts with multiple currencies).django-smalluuid
. UUIDs in URLs will now be rendered in the regular UUID format.django-model-utils
. Model choices have the same values, but the data structures have changed to use the Django-nativemodels.TextChoices
:Account.TYPES
is nowAccountType
TransactionCsvImportColumn.TO_FIELDS
is nowToField
TransactionCsvImport.STATES
is nowTransactionCsvImportState