Jason Held
Jason Held
hi! seems this is still an issue (obviously I'm quite delayed in reviewing this), i imagine. can you try writing up a test? Otherwise agreed, pretty straightforward change.
OK. Doesn't seem pre-save related at this point. But, the issue definitely occurs when all of the latest migrations are applied.
@KonstantinSchubert thank you for the write up. I'll look into it on my end too to verify, but in the meantime, if you have any recommendations for a solution or...
#80 should fix this (at least the bug). We will want to fix it in an algorithm fix, but at least we can guard against the exception.
thank you for this. Can you please add at least one test that utilizes this?
@Etenil I appreciate the effort you've put in. We might merge as is. I had a feeling this might happen. I think I got around it with the delta/changed fields...
Yes. I think I have both a test case (and supporting code in the library which is being tested) for the delta/changed field on/off flag. The setting that's being tested...
@Etenil can you rebase? We may not _require_ an additional test, but want to make sure things are still passing.
Hi. I'm not sure I'm particularly familiar with multi-tenant setups. That said, I don't think (at this time) that easy audit is implemented differently than any other django-app, so I...
Looks like this was before easyaudit upgraded to have django 1.8+ support. If you upgrade to a more recent version I think you'll find that it works.