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

After 11/21: Test redirects merged into 23.2 release #5965

Closed
4 tasks done
Tracked by #138 ...
dorothyyeager opened this issue Oct 19, 2023 · 5 comments
Closed
4 tasks done
Tracked by #138 ...

After 11/21: Test redirects merged into 23.2 release #5965

dorothyyeager opened this issue Oct 19, 2023 · 5 comments
Assignees
Milestone

Comments

@dorothyyeager
Copy link
Contributor

dorothyyeager commented Oct 19, 2023

Summary

What we're after:
We want to make sure our transition and www redirects merged into the 23.1 release work and add any Wagtail or www redirects as needed.

*Redirects include guidance documents search, election results page and Commissioner page. The redirect PRs were made to update existing redirects in place.

Related issues

#5883
fecgov/fec-proxy#378
fecgov/fec-proxy#379
fecgov/fec-proxy#382
fecgov/fec-proxy#383

Completion criteria

  • Redirects done in above PRs tested to ensure they are working (both transition and www).
  • If redirects are not working, do what is needed to fix them (PR or wagtail redirect)
  • Spreadsheet updated thoroughly to note testing and fixes.
  • Testing ticket created for PRs for next sprint.
@dorothyyeager dorothyyeager added this to the Sprint 23.2 milestone Oct 19, 2023
@dorothyyeager dorothyyeager changed the title Test redirects merged into 23.1 release After 11/7: Test redirects merged into 23.1 release Oct 19, 2023
@dorothyyeager dorothyyeager changed the title After 11/7: Test redirects merged into 23.1 release After 11/21: Test redirects merged into 23.2 release Nov 8, 2023
@dorothyyeager
Copy link
Contributor Author

The redirects have been tested. Most work; a few need fixing.

@dorothyyeager
Copy link
Contributor Author

Thanks to Pat, we've spotted some typos on my end that were fixed and those redirects are working. Also have put in a PR for one item that needed to be added to www code.

There's one weird item where a wagtail redirect is not working as expected:
image

@patphongs is looking into this. (It's line 47 on the redirect spreadsheet)

@dorothyyeager
Copy link
Contributor Author

We're going to fix the Form 9 redirect via www as well. The PR has been updated. Once that PR has been merged, this ticket can be closed.

@dorothyyeager
Copy link
Contributor Author

Testing ticket created at #6009

@dorothyyeager
Copy link
Contributor Author

PR merged; this can be closed.

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

No branches or pull requests

3 participants