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

Redirect old calculators to new one and remove old calculators #3783

Closed
1 task done
Tracked by #169
JonellaCulmer opened this issue May 27, 2020 · 7 comments
Closed
1 task done
Tracked by #169

Redirect old calculators to new one and remove old calculators #3783

JonellaCulmer opened this issue May 27, 2020 · 7 comments

Comments

@JonellaCulmer
Copy link
Contributor

JonellaCulmer commented May 27, 2020

What we're after: After the new calculator is deployed we need to implement redirects from the old three calculators, currently on transition. We also need to remove the old calculators.

Old calculators:

  1. https://transition.fec.gov/af/afcalc_20190101.shtml
  2. https://transition.fec.gov/af/afcalc_20171228.shtml
  3. https://transition.fec.gov/af/afcalc_20170101.shtml

There are also some related links and older calculators on transition that should be redirected too. The content team has captured these on its spreadsheet for redirects.

(Eventual) Link to new calculator:
https://www.fec.gov/legal-resources/enforcement/administrative-fines/calculating-administrative-fines/#calculating-a-fine

Completion criteria:

@dorothyyeager
Copy link
Contributor

There's some additional administrative fine files in transition to redirect and take down beyond the ones noted in this comment. Content team has put them into our spreadsheet and will take care of it. Our usual process for redirects is noted in #3827 which is our ticket for tracking non-admin fine redirects that need doing. We plan to do them all in one bunch.

@dorothyyeager
Copy link
Contributor

We'll also need to do www redirects (or update some in some cases). So there are two redirect PRs in progress;
Transition: fecgov/fec-proxy#172
www: fecgov/fec-proxy#173

@dorothyyeager
Copy link
Contributor

The redirect PRs are ready for review by @patphongs

@JonellaCulmer
Copy link
Contributor Author

@dorothyyeager Can this ticket be closed in favor of #3827?

@dorothyyeager
Copy link
Contributor

Hi @JonellaCulmer - no. We're experiencing issues with the redirects this issue is tracking. The other issue's redirects worked just fine and I closed it this AM.

@dorothyyeager
Copy link
Contributor

Have put in a PR to get these redirects in. Once they are merged and tested, we can close this ticket and move on to the take down ticket.

@dorothyyeager
Copy link
Contributor

Redirects merged, deployed, testing and working.

Closing this ticket, and we'll take down the pages in #3843

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

2 participants