ImaginaryLandscape / django-multipage-form

Support for Django ModelForms spanning multiple pages
MIT License
11 stars 2 forks source link

Get this working with Django 5.* #4

Open Anni-g opened 7 months ago

Anni-g commented 7 months ago

I tried to implement this wonderful app in a current environment and figured out there were some problems with Django 5. Making the following changes to views.py seemed to help:

But this would break backwards compatibility, wouldn't it?

nttaylor commented 7 months ago

I think you could replace pytz as you describe without problems. This should work with versions as low as Django 3.2, which is the oldest version that this app supports. As for replacing "self.template_name" with "self.form_template_name", I don't think that should be necessary. As I understand it, those two attributes serve different functions, and there should be no problem with using "template_name" in Django 5.x. However, I have not tested this myself.

Can you make a PR for the "pytz" issue and any others you have found? I'll be happy to take a look.