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

Move statistical data summaries from transition S3 to production S3 #4052

Closed
6 tasks done
johnnyporkchops opened this issue Sep 17, 2020 · 3 comments
Closed
6 tasks done
Assignees
Milestone

Comments

@johnnyporkchops
Copy link
Contributor

johnnyporkchops commented Sep 17, 2020

Summary

As followup to Implement filterable campaign finance data summaries (issue #3943 PR #4088**) we need to move the .PDF and .XLS(X) files to which the new tables link, to the production S3 bucket and maintain the current path after the domain.

Action items

  • Copy statistical data summary .PDF and .XLS(X) files from transition S3 bucket to production content S3 bucket as the followup to Summary data tables #4088
  • Make relevant changes to the paths currently in tables in the Wagtail pages for Congressional. PAC, Party Committee, and Communication Filings
  • export names of paths to maintain SEO and ensure we are not abandoning links
  • decide on what paths to use
    [ ]update paths (regex) in statistical-summary.js after files are added to content s3 bucket Not needed
  • Create redirect for old statistical summary files to new bucket

This is the separate issue for deleting files: #4208

  • Make sure everything transferred OK before deleting.
    • Delete files from transition S3. Confirm this with team.

Since it is priority to take down transition web pages (and not as much, transition bucket), we left these files on transition give us a sooner delivery date for the new web pages, adding the work in this ticket as a followup.

*Note: #4088 is only for current summary data tables, there is another issue with #4020 for moving the archived tables, but it makes sense to move all files at once so when archived tables are moved to Wagtail, they will be ready.

Related issue: #3943
Related PR: #4088

Completion criteria

  • data summary files are copied from the transition s3 to the content s3
@patphongs
Copy link
Member

Redirect PR is in and ready for review by @johnnyporkchops: fecgov/fec-proxy#206

@patphongs
Copy link
Member

Redirects have been merged and will deployed on 12/8.

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