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

Add www redirects for form 1 and former_members paths #379

Merged
merged 5 commits into from
Oct 31, 2023

Conversation

dorothyyeager
Copy link
Contributor

@dorothyyeager dorothyyeager commented Oct 19, 2023

  • Redirects www.fec.gov/pdf/fecfrm1 to its new URL at https://www.fec.gov/resources/cms-content/documents/policy-guidance/fecfrm1.pdf
  • Redirects /members/former_members/... to /resources/about-fec/commissioners/....
  • Temporarily disable incorrect redirect for non-existent policy-guidance/documents
  • Fixes incorrect redirect for national-party-committee-accounts:
    ^/updates/fec-issues-interim-reporting-guidance-for-national-party-committee-accounts/ /updates/guidance-search/fec-issues-interim-reporting-guidance-for-national-party-committee-accounts/

How to test:

@johnnyporkchops
Copy link
Contributor

@dorothyyeager , Do we want to correct that broader, incorrect redirect for PDF forms too?
rewrite ^/pdf/forms/(.*) https://www.fec.gov/resources/cms-content/policy-guidance/documents/$1 redirect;

@johnnyporkchops johnnyporkchops changed the title Add www redirect for form 1 path Add www redirects for form 1 and former_members paths Oct 22, 2023
@dorothyyeager
Copy link
Contributor Author

dorothyyeager commented Oct 23, 2023 via email

@johnnyporkchops
Copy link
Contributor

johnnyporkchops commented Oct 25, 2023

@dorothyyeager Why are we redirecting the original /updates/.. pages to their alias in /guidance-search? The pages are always identical so there is no reason to redirect unless you want the user to see a certain path in the URL and if that's the case, shoudn't we be redirecting the alias back to the original? In other words, if a user finds a page in the policy-guidance search results with a URL containing /updates/guidance-search/..., when they click on it, it brings them to the page in its original location among all the other /updates pages.

# Redirects for /updates/

@dorothyyeager
Copy link
Contributor Author

dorothyyeager commented Oct 25, 2023 via email

@johnnyporkchops
Copy link
Contributor

johnnyporkchops commented Oct 25, 2023

We don’t want two identical pages. We only want one of each. The plan is to delete the old ones once redirects are working.

@dorothy, We do not have two identical pages, we have one page with an alias in another location. To reiterate, the Wagtail aliased page automatically matches the original and the editor never has to access it or edit the alias. Another reason we chose this setup is that if the Policy guidance setup changes or we end up discontinuing that requirement, then we do not have to keep moving pages and putting in redirects (technical debt). I probably should have demoed the alias function in Wagtail rather than just explaining it. Perhaps I can do that as part of this discussion.

Also, If we were to delete the original page from /updates and move it to /guidance-search/, then it would no longer show up when someone filtered for it on the /updates page.

I am also looking at a way to sync the policy pdfs to the policy-guidance directory (not just copy them)... again so that content does not get moved all over the place and cause us linking and redirect headaches... all for the godforsaken policy-guidance requirement 😃

@kathycarothers kathycarothers merged commit 3d5a019 into develop Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix broken links and formatting on Commissioner bio pages
3 participants