Skip to content

Conversation

PetrDlouhy
Copy link
Collaborator

…reak dj-money

@PetrDlouhy PetrDlouhy marked this pull request as ready for review January 11, 2023 17:21
@PetrDlouhy
Copy link
Collaborator Author

The migration counts with migrations from #76 - will need to be merged first or the migration recreated.

@nitsujri
Copy link
Contributor

nitsujri commented Feb 16, 2023

Linking Issue #78. Thanks so much for this fix!!

@PetrDlouhy can we remove dependency on #76 as it looks like it'll be a while before that one can get merged in. Would love to officially have this warning removed.

@PetrDlouhy PetrDlouhy merged commit 5c1d219 into master Feb 17, 2023
@PetrDlouhy
Copy link
Collaborator Author

@nitsujri I did change the PR to not change default value of CURRENCIES and thus not creating migration at all and avoiding the dependency.
I hope that is enough, you can still set HORDAK_CURRENCIES in your settings.

I will release this now.

@adamcharnock adamcharnock deleted the fix_currencies branch June 28, 2024 09:11
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