Bogdanp / django_dramatiq

A Django app that integrates with Dramatiq.
https://dramatiq.io
Other
331 stars 77 forks source link

Support Django's settings to set message encoder #17

Closed beigna closed 5 years ago

beigna commented 5 years ago

Hi @Bogdanp based on the https://github.com/Bogdanp/dramatiq/issues/17 I made this little contribution to make more "Djangish" your package. Now you can set a custom Encoder via Django's settings.

I hope that the naming convention that I applied be right (I thought that the encoder is related to the broker's options but ... you may decide this.) Thanks for your time!

beigna commented 5 years ago

@Bogdanp I think that I messed up the PR. Please check the changes :)

Thanks for your time!

Bogdanp commented 5 years ago

Thanks again, @nachopro ! I merged your changes with rebase, added you to the contributors file and released version 0.4 with these changes.