August Raack
August Raack
@hramezani @Linkid Thanks for the review. I added a migration and updated the changelog. Sorry about the missing migration! I'm not experienced in contributing to Django packages with migrations. I...
@hramezani I noticed in the migration file that you suggested it changed the `null=True` to `blank=True` should this be changed on the model too? I can understand if you'd like...
@hramezani I created 502,000 `LogEntry` records using a fresh installation of `auditlog` in a development environment running our project. I am using a local Postgres instance running in docker. All...
Thanks everyone for the collaboration! I felt like this was a natural progression to the code and fits within the vision for your project. I'm glad you guys agree! I'm...
This PR contains a migration. If the wonderful maintainers of django-auditlog would help my team out tremendously by adding this to their project, we would need to be mindful of...
Given some of the feedback here, I've made a few adjustments. I'm still optimistic that you all may still be favorable to including the `serialized_data` field in the auditlog project....
> To me, the change has two parts > > 1. Adding a JSON field to store the JSON repr of the object. (sounds good) > 2. The `HistoricalStateLookupManagerMixin` class,...
@hramezani I really apologize for that. I had some git madness on my end and unintentionally made that happen. It's fixed now. I was hoping it would be fixed before...
@hramezani @alieh-rymasheuski Thank you for the code review. I appreciate your feedback and I've made effort to make sure I have addressed each of your comments. There are a few...
@hramezani Before you guys merge. I'm looking into something that's coming up with implementing this in our system. The issue occurs if datetimes are written as strings on in-memory model...