Not Read: Publication Import

[buttonwillowsix]

Adding export view, and possibly too much

NOT READY FOR REVIEW

Summary

• Creates a table view available only to admins from which publication data can be copy-pasted into a spreadsheet using our import format

Review By (Date)

• No current deadline

Criticality

• Would affect build of 3-5 Precourt Sites on SDSS

Urgency

• Hard to say

Review Tasks

Setup tasks and/or behavior to test

1. Check out this branch
2. Make sure you have some publication data
3. Go to /admin/publications/export
4. Copy the test out of that view and into a blank spreadsheet
5. Export the CSV
6. Upload onto another site (or make changes and import onto the same site) using our Publications importer

Site Configuration Sync

• Yep. That is all there is.

Front End Validation

• Design is approved by @ user?
• HTML validation: Is the markup using the appropriate semantic tags and passes validation? Or, QA request ticket created?
• Cross-browser testing: Has been performed? Or, QA request ticket created?
• Automated accessibility: Scans performed? Or, QA request ticket created?
• Manual accessibility: Manually tested? Or, QA request ticket created?

Backend / Functional Validation

Code

• Are the naming conventions following our standards?
• Does the code have sufficient inline comments?
• Is there anything in this code that would be hidden or hard to discover through the UI?
• Are there any code smells?
• Are tests provided? eg (unit, behat, or codeception)

Code security

• Are all forms properly sanitized?
• Any obvious security flaws or new areas for attack?

General

• Is there anything included in this PR that is not related to the problem it is trying to solve?
• Is the approach to the problem appropriate?

Affected Projects or Products

• SDSS
• Publications

Associated Issues and/or People

• Probably needs a ticket

Resources

• AMP Tool
• Accessibility Manual Test Script
• HTML Validator
• Browserstack and link to Browserstack Credentials