fecgov / fec-cms

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

QA Wagtail V3 and log any issues #5519

Closed patphongs closed 1 year ago

patphongs commented 1 year ago

Summary

What we're after: We need to do quality assurance testing and log any issues for the new version of Wagtail V3. It has been deployed to feature space for testing.

Related issues

List any relevant related issue(s)

Tech steps or considerations

Take a look at the "What's new" section of the Wagtail documentation. Here's a summary of some things to test:

Completion criteria

Future work

Will roll out v3 to Production, then move to v3>v4

patphongs commented 1 year ago

Wagtail search is not working

Screen Shot 2022-12-14 at 11 03 33 AM
johnnyporkchops commented 1 year ago

@patphongs, for the admin search, could this be the issue: . (we ignored it in the bump to 2.15 because we dont have it in installed apps and I think it comes into play in V3) https://docs.wagtail.org/en/stable/releases/2.15.html#database-search-backends-replaced

kathycarothers commented 1 year ago

I am moving this ticket into the next sprint

patphongs commented 1 year ago

for the admin search, could this be the issue: . (we ignored it in the bump to 2.15 because we dont have it in installed apps and I think it comes into play in V3)

Thanks for this tip @johnnyporkchops! Once I made this modification, the wagtail search worked again

patphongs commented 1 year ago

@kathycarothers A good video to watch for changes to the Wagtail 3 version is here: https://www.youtube.com/watch?v=owsdUze9m0Q. It'll show you the improvements made for this version, so you can kind of know what to look out for.

patphongs commented 1 year ago

I found that on the services landing page template, it requires something in the intro block to be published. But this is not related to the Wagtail 3 upgrade as this is also a requirement in production. I tested inserting text into the intro block and because we don't have it being inserted anywhere on the page, you don't see it. I think we can just move past this for now, if it needs fixing at a later date, we can remove the intro block from there.

patphongs commented 1 year ago

Image duplication detection works great

Screen Shot 2023-01-11 at 2 43 03 PM
patphongs commented 1 year ago

Wagtail v3 has been fully tested, all aspects seem to be functioning well. Will review the PR with @rfultz and @johnnyporkchops tomorrow to make sure things are looking good.

djgarr commented 1 year ago

@patphongs asked me to test the page title updates. They are functioning as they do in V2: to change the internal page title, the page needs to be published first

pkfec commented 1 year ago

PR https://github.com/fecgov/fec-cms/pull/5517 got merged. Closing this issue.