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

Pages getting 500, server-error when trying to edit in Wagtail #3383

Closed
1 task done
Tracked by #138
johnnyporkchops opened this issue Nov 24, 2019 · 7 comments
Closed
1 task done
Tracked by #138
Assignees
Milestone

Comments

@johnnyporkchops
Copy link
Contributor

johnnyporkchops commented Nov 24, 2019

Summary

The following pages in /press return a 500, server error when trying to open for editing in Wagtail:

Also, more pages than not in About/Reports about FEC/Modernizing the E-filing experience and infrastructure return a 500, server error when trying to open for editing in Wagtail:

Completion criteria

@JonellaCulmer JonellaCulmer added this to the PI 11 planning milestone Nov 25, 2019
@johnnyporkchops johnnyporkchops changed the title Some Press pages getting 500, server-error when trying to edit in Wagtail Press and Reports About FEC pages getting 500, server-error when trying to edit in Wagtail Nov 26, 2019
@dorothyyeager
Copy link
Contributor

The Press Office is no longer updating the one page about public funding - it's been retired and redirected to our main page on it. So you might not have to fix that one.

@JonellaCulmer
Copy link
Contributor

JonellaCulmer commented Dec 10, 2019

@johnnyporkchops Do we still need to address this work? If not, please close.

Update: Talked to @johnnyporkchops and this work is still necessary.

@dorothyyeager
Copy link
Contributor

Do we want to put it into a milestone? (Yes, definitely necessary, and the page @kathycarothers found on filing amendments should be added to list above.) @JonellaCulmer

@JonellaCulmer JonellaCulmer changed the title Press and Reports About FEC pages getting 500, server-error when trying to edit in Wagtail Pages getting 500, server-error when trying to edit in Wagtail Dec 13, 2019
@kathycarothers
Copy link
Contributor

I have added the page to the ticket.

@pkfec
Copy link
Contributor

pkfec commented Jan 13, 2020

For some wagtail pages, even after removing the unwanted tags <span> from the body of the text and try to save the page, i ended up with an validation error .
ERROR:

page.save()
Traceback (most recent call last):
File "", line 1, in
File "/home/vcap/deps/0/python/lib/python3.6/contextlib.py", line 52, in inner
return func(*args, **kwds)
File "/home/vcap/deps/0/python/lib/python3.6/site-packages/wagtail/core/models.py", line 443, in save
self.full_clean()
File "/home/vcap/deps/0/python/lib/python3.6/site-packages/wagtail/core/models.py", line 433, in full_clean
super().full_clean(*args, **kwargs)
File "/home/vcap/deps/0/python/lib/python3.6/site-packages/django/db/models/base.py", line 1250, in full_clean
raise ValidationError(errors)
django.core.exceptions.ValidationError: {'menu_title': ['This field cannot be blank.']}

@pkfec
Copy link
Contributor

pkfec commented Jan 13, 2020

The new model requires that the menu_title should be set on the page if doesn't exist already before saving the page.

Paired with @johnnyporkchops and fixed above 4 wagtail pages in dev environment first. Tested and make sure the links are editable in wagtail.

Copied the changes from dev interactive shell to the prod space. Wagtail pages are now editable in Dev and Prod spaces.

@pkfec pkfec closed this as completed Jan 13, 2020
@patphongs
Copy link
Member

@pkfec @johnnyporkchops Nice work on pairing with this ticket! Thanks for fixing these pages!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants