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 field_access_terms to media migrations & media tests #110

Closed
12 of 13 tasks
bseeger opened this issue Apr 7, 2021 · 1 comment
Closed
12 of 13 tasks

Add field_access_terms to media migrations & media tests #110

bseeger opened this issue Apr 7, 2021 · 1 comment

Comments

@bseeger
Copy link

bseeger commented Apr 7, 2021

The Access Control PR (Role configuration) and the Media migrations happened sort of back to back. The Access Control PR doesn't account for field_access_terms in media migrations because the migrations weren't finished yet.

This ticket is to complete that work by:

  • Migrations: adding field_access_terms to all the idc media migrations in codebase/config/sync
    • audio
    • document
    • extracted text
    • file
    • fits no fits migration
    • image
    • remote video
    • video
  • Verification Tests:
    • update the csv files for media migrations to include field_access_terms in them
    • update files containing expected json
    • update the verification test to include checking for values in field_access_terms
@bseeger
Copy link
Author

bseeger commented Jul 16, 2021

Resolved with: Address field_access_terms tickets and a few minor things #178

@bseeger bseeger closed this as completed Jul 16, 2021
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