Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

A bunch of work and improvements #58

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

matiasherranz
Copy link

Hi!

I've took some time to improve the (already awesome) django-notification app.

What have I done?

  • Made it work with django 1.4
  • [this is a big one] Made django-notification timezone aware and compliant.
  • Made every single file PEP8 compliant
  • Fixed a bunch of issues I found while going through the codebase
    • Quite a few code improvements.

I think you may find my changes make sense, and I gave it quite a few hours of work, so, why not a pull request for the sake of improving django-notification? :)

Best regards,

Matías Herranz [matiasherranz@gmail.com]

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant