django-payfast

django-payfast copied to clipboard

Support for `python_2_unicode_compatible` decorator removed in Django 3

1

@PiDelport I have tested this simple patch in my project successfully (Python version 3.6.6 and Django==3.0.2). I have however **not run the test suite** nor have I removed the use...

atlasstrategic

Please see Pull Request for details https://github.com/PiDelport/django-payfast/pull/18

atlasstrategic

Hi @PiDelport :wave: I've added some changes for Django 4.0 support. Got it working on my side, tests passing OK. Feel free to make any additional changes. Thanks!

atlasstrategic

tafadzwatazvitadza

Django 3.0 Support

3

Django 3.0 [removes](https://docs.djangoproject.com/en/3.0/releases/3.0/#removed-private-python-2-compatibility-apis) `django.utils.encoding.python_2_unicode_compatible()`, which is used as a [decorator](https://github.com/PiDelport/django-payfast/blob/9c14aec3d3993644464c1cdacc89614500de664d/payfast/models.py#L11) on the `PayFastOrder` model. Removing this decorator is required for Django 3.0 support. ## Reference - https://github.com/macropin/django-registration/issues/371

atlasstrategic

Hi everyone, Whats the process for using subscription based payment methods? Create a subscription in payfast then does Django-payfast allow you to pull the subscriptions from payfast and migrate to...

ShaunDeponte

notify_handler should rather 500 on an error

2

The code returns a 400 if the form validation fails. Since the notify POST from Payfast usually happens out of band no one is around to see these errors, except...

hedleyroos