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 new properties to Images work type #184

Merged
merged 1 commit into from
Nov 20, 2023
Merged

Conversation

kirkkwang
Copy link
Contributor

@kirkkwang kirkkwang commented Nov 17, 2023

Story

This commit will add new properties to the Images work type. The new properties are:

additional_rights_info
institution
resource_type
types
format
video_embed

Ref:

Expected Behavior Before Changes

Image work type was just had the default metadata

Expected Behavior After Changes

Image work type has customized metadata

Screenshots / Video

Form

image

Record

image

index.as :stored_searchable, :facetable
end

property :types, predicate: ::RDF::URI.new("https://atla.com/terms/types")
Copy link
Contributor

Choose a reason for hiding this comment

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

Why are there two :types predicates?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

not really sure, the other work types also have this so i was following the pattern

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added the comment that I see on the other models where it's defined twice, also asked the team about it here:
https://assaydepot.slack.com/archives/C030UPFB8LW/p1700258300609389

This commit will add new properties to the Images work type. The new
properties are:

  additional_rights_info
  insitution
  resource_type
  types
  format
  video_embed

Ref:
  - #145
@kirkkwang kirkkwang merged commit 878ef88 into main Nov 20, 2023
4 of 7 checks passed
@kirkkwang kirkkwang deleted the i145-image-work-type branch November 20, 2023 18:47
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.

2 participants