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

Validation and field updates #208

Merged
merged 9 commits into from
Sep 10, 2021
Merged

Validation and field updates #208

merged 9 commits into from
Sep 10, 2021

Conversation

birkland
Copy link

@birkland birkland commented Aug 18, 2021

  • Makes media_of a required field. Ingests will no longer silently orphan media if the media_of field is
    empty, or malformed (i.e. a typo, pointing to a non-existent id).
  • Makes field_digital_publisher required, as per stakeholder feedback related to citations
    • Multiple test data tweaks to accommodate this requirement
  • Increases the length of alt title to 5000 characters.

Resolves jhu-idc/iDC-general#406

@github-actions
Copy link

This PR has no dependency differences with the base branch

@birkland birkland changed the title [experimental] Require media_of for all media Validation and field updates Sep 8, 2021
@github-actions
Copy link

github-actions bot commented Sep 8, 2021

This PR has no dependency differences with the base branch

2 similar comments
@github-actions
Copy link

github-actions bot commented Sep 9, 2021

This PR has no dependency differences with the base branch

@github-actions
Copy link

github-actions bot commented Sep 9, 2021

This PR has no dependency differences with the base branch

@github-actions
Copy link

github-actions bot commented Sep 9, 2021

This PR has no dependency differences with the base branch

@birkland birkland requested a review from bseeger September 10, 2021 12:35
Copy link

@bseeger bseeger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There might be a need to change the media_of fields to accept collection_object as well. Testing this out now.

@github-actions
Copy link

This PR has no dependency differences with the base branch

@bseeger
Copy link

bseeger commented Sep 10, 2021

Spinning this up now to give it a look.

Copy link

@bseeger bseeger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me and works as advertised.

Worth nothing -- this ticket makes it so one can only attach Images to Collections. One cannot attach other Media Types to Collections anymore, after this goes in.
(I think this is sufficient, btw, but just wanted to get it in writing somewhere).

@birkland birkland merged commit 37d9ccc into development Sep 10, 2021
@birkland birkland deleted the require_media_of branch September 10, 2021 21:15
jaredgalanis pushed a commit that referenced this pull request Feb 15, 2022
…gs-non-writable

Ensure drupal settings are writable to fix non-bootable container.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Data Model Configuration Tweaks
2 participants