Seems like a good idea to me. I would squash your commits anyway, so there's no need for submitting a new PR. You could also squash yourself. 

However do you still need the change you're proposing, as you already changed your templates? 

Anyway thanks for contributing and If you like you could also improve the app in other areas.

> Op 25 nov. 2013 om 07:46 heeft Christian Hammond <notifications@github.com> het volgende geschreven:
> 
> In two_factor/views/core.py:
> 
> > @@ -115,6 +117,12 @@ def get_context_data(self, form, **kwargs):
> >                  phone for phone in backup_phones(self.get_user())
> >                  if phone != device]
> >          context['cancel_url'] = settings.LOGOUT_URL
> > +
> > +        redirect_field_name = self.kwargs.get('redirect_field_name',
> > +                                              REDIRECT_FIELD_NAME)
> > +        context[redirect_field_name] = \
> > +            self.request.GET.get(redirect_field_name, '')
> Nope, getting rid of them.
> 
> Given the removal of the <input/> in the template, the purpose of this change really becomes providing a custom field for the redirect URL. In our project, we had a custom template and it was doing an explicit action="<url>", rather than action="". Based on your reviews, I've changed our template.
> 
> So, if you like, I can submit a new pull request focusing entirely on the addition of redirect_field_name (plus the two unit tests), to keep the history clean.
> 
> —
> Reply to this email directly or view it on GitHub.