fecgov / fec-cms

The content management system (CMS) for the new Federal Election Commission website.
https://www.fec.gov
Other
96 stars 38 forks source link

[DUE 4/15/2021] Testing Wagtail v2.11.6 and Django v3.1 upgrades #4529

Closed rfultz closed 3 years ago

rfultz commented 3 years ago

Summary

Wagtail v2.11.6 and Django v3.1 will be deployed to the feature environment for testing. Testing ticket to put the new versions of Wagtail and Django through their paces.

Notes

The related ticket and PR

Details

We're upgrading Wagtail by several versions and Django by a major + minor version and there were some breaking changes.

To Test

Wagtail login and Cookie testing

Do not clear cookies before starting this test.

How to check cookies (not needed, but if you want to)

One of the breaking changes has to do with how the system encodes and decodes session IDs so we want to make sure we start with cookies from older log-ins. (If you hadn't already had a cookie from the older version, there's no way to fake it without deploying the older (current) version again.) FTR, to check your cookie:

Check all templates working (Jonella, coordinating template reviews)

Check pages outside of Wagtail (Jonella)

Check functionality of Wagtail in general (everyone, while checking other templates)

Check responsive object (John)

Have fun breaking things! (always better to break in feature than have problems with Prod)

Any cool new Wagtail functionality from upgrade?

patphongs commented 3 years ago

Cool new stuff I just found. I haven't had a chance to play around with it yet though.

Screen Shot 2021-04-01 at 4 49 10 PM Screen Shot 2021-04-01 at 4 48 58 PM
rfultz commented 3 years ago

Yep!

Here are the various release notes if you're interested: Release notes for Wagtail 2.8 Release notes for Wagtail 2.9 Release notes for Wagtail 2.10 Release notes for Wagtail 2.11

Release notes for Django 3.0 Release notes for Django 3.1

patphongs commented 3 years ago

We've decided to push the upgrade to the 14.4 release which will be on 4/27/2021. This will give us more time to test and deploy after the April quarterly filings are due.

dorothyyeager commented 3 years ago

Checks of the home page banner and emergency alert banner went well. Banners appeared when they were supposed to, appeared correctly, and disappeared when they were supposed to. Here's screencaps of each.

banner-test-ok emergency-banner-test
djgarr commented 3 years ago

Ran into some shenanigans when testing the full width page template

image

The table i created showed up correctly below the above image

Update: ran into the same issues as others when trying to use the glossary function

image

djgarr commented 3 years ago

The example page didn't work. not sure if there's an issue uploading an image, but I tried and got this result

image

patphongs commented 3 years ago

Wagtail icon that appears on the page when you're logged in and want to edit the page seems to have disappeared from the circle. The icon is definitely there, but the classes look missing: wagtail-icon wagtail-icon-wagtail.

Screen Shot 2021-04-06 at 4 01 44 PM Screen Shot 2021-04-06 at 3 58 34 PM
patphongs commented 3 years ago

Any page revisions, now called "history", made prior to the upgrade are now missing. I tried the "History" button to look at the About landing page history and I don't see any history prior to the upgrade at all. I should see at least a dozen revisions for the About page.

Screen Shot 2021-04-06 at 4 09 30 PM

I tried to save a draft on the page just now and all I see is my most recent revisions after the upgrade.

Screen Shot 2021-04-06 at 4 14 10 PM
dorothyyeager commented 3 years ago

Custom page template issues - test page is at https://fec-feature-cms.app.cloud.gov/help-candidates-and-committees/test-of-custom-page-4-2021/ (https://fec-feature-cms.app.cloud.gov/admin/pages/11622/edit/)

1. Lists

Numbering lists in the rich text editor do not work. Numbers don't render and the numbering when indented didn't change from 1, 2, 3.

image

Addition from Pat for unordered lists The bullets do not appear for unordered lists on the custom page and text font size is smaller. Example link.

Screen Shot 2021-04-06 at 4 49 25 PM

2. Links:

3: Images

image

4. Snippets: There's no padding around the snippet when inserted. (This is needed)

5. Superscript: Trying to apply superscript to text caused the content to crash. image

6. Sidebar We only need paragraph and heading.

7. Related Topics - Box size doesn't look correct and words should be centered. Box should be below the Citations box. image

patphongs commented 3 years ago

Not a bug but just noting that there is a difference in the page status display in the newer Wagtail version.

A page is in draft and not published live: It will display the time saved of the latest draft and not have a live button displayed in the page.

Screen Shot 2021-04-06 at 4 19 15 PM

A page is published and live: It will display the time the page was published live and the "Live" button on the right hand side of the page.

Screen Shot 2021-04-06 at 4 19 35 PM

A page is published and has a saved draft that is not published: It will display the time of the last saved draft but also indicate that the page is live with the button on the right.

Screen Shot 2021-04-06 at 4 21 29 PM
djgarr commented 3 years ago

On the Example page, the numbering and bullet points do not work

image

kathycarothers commented 3 years ago

When testing the Commissioner page template I went into the paragraph box to add text. For one of the words I tried to link it to a glossary term and the page crashed. This also happened when I tried to add an anchor and Sans Serif font options too. See the picture below:

image

And the numbering, bullets points, and the horizontal line did not work either:

image

kathycarothers commented 3 years ago

Digest pages problems:

image

Missing horizontal line:

image

patphongs commented 3 years ago

No issues with Wagtail login in feature space after logging in on the old version and then logging in on the new version.

dorothyyeager commented 3 years ago

Press Release page template: https://fec-feature-cms.app.cloud.gov/updates/test-press-release-page-4-2021/

No notable issues. I did not fully test the rich text paragraph editor since issues with that were noted in the custom page template comment. I did notice the bullets worked in this template.

rfultz commented 3 years ago

I've created an initial MoSCoW tracker here. Trying to keep things organized and sortable but still conversational shouldn't be such a challenge https://docs.google.com/spreadsheets/d/17z1ks8BHjFVUls-nVf5a8JaE-HFCvOXBZVE2aJ0eikU/edit?usp=sharing

JonellaCulmer commented 3 years ago

Resource page:

JonellaCulmer commented 3 years ago

Custom page addition:

Appearance on .gov:

Screen Shot 2021-04-12 at 8 53 57 AM

Appearance on feature:

Screen Shot 2021-04-12 at 8 54 05 AM
rfultz commented 3 years ago

The example page didn't work. not sure if there's an issue uploading an image, but I tried and got this result

image

David, how's this? image

rfultz commented 3 years ago

Ran into some shenanigans when testing the full width page template

image

The table i created showed up correctly below the above image

Update: ran into the same issues as others when trying to use the glossary function

image

@djgarr is this still an issue if we add a </table>?

rfultz commented 3 years ago

Custom page template issues - test page is at https://fec-feature-cms.app.cloud.gov/help-candidates-and-committees/test-of-custom-page-4-2021/ (https://fec-feature-cms.app.cloud.gov/admin/pages/11622/edit/)

1. Lists

Numbering lists in the rich text editor do not work. Numbers don't render and the numbering when indented didn't change from 1, 2, 3.

image

Addition from Pat for unordered lists The bullets do not appear for unordered lists on the custom page and text font size is smaller. Example link.

Screen Shot 2021-04-06 at 4 49 25 PM

2. Links:

  • Anchor link: Creating an anchor link in the rich text editor paragraph block caused the page to crash and lose unsaved work. image
  • Glossary link: Trying to add one caused the content to crash and lose unsaved work. image

3: Images

  • Thumbnail images inserted into a page's paragraph block work.
  • The image block - used for larger or medium sized images on a page (like webinar ads) did not render as a separate block.
  • We'd also want to ensure our current padding is still intact on image blocks. For example, an html block added right as an image block should've had some padding between the html and the image, but instead it didn't put the padding there, it looks like it padded in the middle of the html:

image

4. Snippets: There's no padding around the snippet when inserted. (This is needed)

5. Superscript: Trying to apply superscript to text caused the content to crash. image

6. Sidebar We only need paragraph and heading.

7. Related Topics - Box size doesn't look correct and words should be centered. Box should be below the Citations box. image

@dorothyyeager do these look right now? image

rfultz commented 3 years ago

@kathycarothers can you point me to where there's an <hr> that works? I'm not seeing it on the live site, either.

rfultz commented 3 years ago
  • Numbered and bulleted lists are working in wagtail editor

@JonellaCulmer I think these have been fixed now. ?

djgarr commented 3 years ago

Ran into some shenanigans when testing the full width page template image The table i created showed up correctly below the above image Update: ran into the same issues as others when trying to use the glossary function image

@djgarr is this still an issue if we add a </table>?

That seems to do the trick. No shenanigans then, just user error. Thanks!

Update: While that did remove the image, the drop down menu on the chart isn't working. I'm not sure if that's because it's a partial list of states and territories, but if you were to choose from one of the states listed on the menu, the chart stays the same

patphongs commented 3 years ago

Testing is complete, all issues have been resolved. Merged to develop and will do additional testing.