Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Apply for Sponsorship template shows Box form #4

Open
jefftriplett opened this issue Mar 16, 2016 · 4 comments
Open

Apply for Sponsorship template shows Box form #4

jefftriplett opened this issue Mar 16, 2016 · 4 comments
Labels

Comments

@jefftriplett
Copy link
Contributor

This kept us guessing for a while, but when a sponsor attempt to apply they see the page's box form instead of the sponsorship form. I fixed it by disabling the editable form for DCUS, but I wanted to report it upstream in case there's a better fix.

Before:

After:

@ossanna16
Copy link
Contributor

Thanks for reporting this @jefftriplett! Any ideas @paltman?

@ossanna16 ossanna16 added the bug label Mar 17, 2016
@paltman
Copy link
Contributor

paltman commented Mar 17, 2016

Looking into this now

@paltman
Copy link
Contributor

paltman commented Mar 17, 2016

There is some sort of conflict going on here, I suspect the {% box %} template tag is overwriting the form variable in the template. The idea for this page is staff being able to edit content on the page as a pre-amble to the form.

@paltman
Copy link
Contributor

paltman commented Mar 17, 2016

Three possible fixes:

  1. Remove the box tag so form isn't overridden (like what you did @jefftriplett)
  2. Update pinax-boxes so that the form that is returned/rendered as part of the tag is named something more explicit so as to have a lower change of collision (e.g. boxes_form)
  3. Update the sponsor view to rename the form to something specific (e.g. sponsor_form)

I think the best solution is option 2, perhaps in addition to option 3, but option 1 is certainly the fastest and best work around in the short term.

euclidian pushed a commit to euclidian/PinaxCon that referenced this issue Feb 17, 2017
Added django compress static files finder in prod.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants