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

Update www redirects with new URLs for documents that are in guidance search and for citizens page #5883

Closed
7 tasks done
Tracked by #173
dorothyyeager opened this issue Aug 31, 2023 · 5 comments
Closed
7 tasks done
Tracked by #173

Comments

@dorothyyeager
Copy link
Contributor

dorothyyeager commented Aug 31, 2023

Summary

What we're after:
We want to make sure we are redirecting users correctly who might have the old URL for guidance search documents or for the citizens page

Related issues

fecgov/fec-proxy#365

Redirect spreadsheet: https://docs.google.com/spreadsheets/d/1vzN-wZlS8K8ZyfExSV3hwXDenDEN9KyyAUJFTI6OjKU/edit#gid=687896915

Completion criteria

  • Tab in redirect spreadsheet set up for tracking these redirects and testing
  • List of guidance search documents new URLs checked against Wagtail redirects and any Wagtail redirects are updated as needed
  • List of guidance search documents new URLs checked against transition redirects and any transition redirects are updated as needed
  • Redirects to old citizens page URL are updated to point to https://www.fec.gov/introduction-campaign-finance/election-results-and-voting-information/
  • Testing ticket created for the transition redirect PR to be merged
  • Wagtail redirects tested after they are updated.
  • Remove redirects to /updates/guidance-search/ since they are not needed.

Future work

Testing ticket

@dorothyyeager dorothyyeager added this to the Sprint 22.6 milestone Aug 31, 2023
@dorothyyeager dorothyyeager changed the title Update Wagtail and Transition redirects with new URLs for documents that are in guidance search Update Wagtail and Transition redirects with new URLs for documents that are in guidance search and for citizens page Oct 18, 2023
@dorothyyeager
Copy link
Contributor Author

PR submitted to update the transition redirect code. Quite a few of them needed updating to match the new URLs.

@dorothyyeager
Copy link
Contributor Author

Two PRs have been submitted:
fecgov/fec-proxy#378
fecgov/fec-proxy#379

Once they are merged, this ticket can be closed.
Testing ticket is at #5965

@dorothyyeager
Copy link
Contributor Author

I've set up a meeting on Wednesday at 3:00 (a rare time that was free on all calendars) for all of the content team, @johnnyporkchops @JonellaCulmer @AmyKort to discuss the guidance search documents and the various comments and confusion about alias and redirects, plus agree on our understanding of what URL will be the official URL for a guidance-search document or page. The outcome of that meeting will determine the outcome of this ticket and related future tickets.

@dorothyyeager dorothyyeager changed the title Update Wagtail and Transition redirects with new URLs for documents that are in guidance search and for citizens page Update www redirects with new URLs for documents that are in guidance search and for citizens page Nov 8, 2023
@dorothyyeager
Copy link
Contributor Author

We had a good meeting today. Outcome:

  • Updates pages that are in guidance search will continue to use the updates URL (guidance-search version is an alias).
  • PDF files in guidance search will use the documents/policy-guidance URL.
  • Dorothy will update redirect PRs to make a few tweaks along these lines and make sure end users get to what we want them to get to. (We'll just do this in this ticket; PRs are ready for review.)
  • We will hopefully get rid of guidance search one day but we need to see what the current request for comments brings.. But it is duplicative and not used much.
  • For now we do need to have John or Robert swap out the PDFs of the guidance search when updated. Pat will look into automating this so we can go back to the old way of just uploading to CMS. The automation would put the updated file into the documents/policy-guidance/ folder in S3.

Once the PRs are merged, this ticket can be closed.

@dorothyyeager
Copy link
Contributor Author

Related PR merged; closing

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

4 participants