Andrew Cornforth

+1 for dark theme ; +1,000,000 for custom stylesheet :)

@natewiebe13 I tested this on postgres with IDENTITY strategy, and everything seems to behave as expected, it also when generating migrations, didn't drop the DEFAULT expression on the id column....

Fixed (theoretically) in #93

Hi @natewiebe13 , Yes, i can rebase onto 1.0 and tweak my submission based on your excellent notes. Also happy to add tests and UA field as part of this...

@natewiebe13 It's rebased ok, but haven't managed to add any tests yet. I have added logging of UA string, as requested, but I'm not convinced it's a good thing to...

I noticed too that you re-added support for dbal ^2.9, that is very appreciated, If i can be of any help getting over the finish line with your 1.0 release....

I think if any more configuration keys are added, it may be better to group them into arrayNodes and reduce the number of setter methods needed in AuditSubscriber, or inject...

found an issue when pulling this branch in to a project, Im pretty sure this has nothing to do with my submitted changes, looks like an unintended consequence of the...

@natewiebe13 I changed AUTO strategy to IDENTITY, (should work on both postgres and mysql. and on Postgres doesn't cause a migration to be created that drops the DEFAULT NEXTVAL('audit_logs_id_seq'). (Fixes...

Anyone looking for a way to workaround this you can just add ```sass .province-container.field > label.required:after display: inline-block vertical-align: top margin: -0.2em 0em 0em 0.2em content: '*' ``` to your...