search

pinax / pinax-messages

a Django app for allowing users of your site to send messages to each other
MIT License
202 stars 58 forks source link

remove all dependency on DUA #40

Closed volksman closed 6 years ago

paltman commented 6 years ago

We should continue to support DUA, but make it optional.

For example, we do this in pinax-stripe:

https://github.com/pinax/pinax-stripe/blob/bf93fe374e03861a965d3a129eb9bdb189eee7ca/pinax/stripe/mixins.py#L6-L9

This has the benefit if we are using django-user-accounts this app will continue to work nicely but if you are not you can gracefully fall back to Django's internals.

volksman commented 6 years ago

@paltman updated!

paltman commented 6 years ago

Thanks!

grahamu commented 6 years ago

Nice fix @volksman