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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Spike: add derived_members and derived_files override to app/presenters/work_show_presenter #34

Closed
2 tasks
Tracked by #19
kirkkwang opened this issue Dec 14, 2022 · 6 comments
Closed
2 tasks
Tracked by #19
Assignees

Comments

@kirkkwang
Copy link
Contributor

kirkkwang commented Dec 14, 2022

Attempt to answer why these methods exist. Is it necessary to keep or transfer over to iiif_print?

  • is it related to this?

Next Steps

Close this ticket if these methods are not required, or after creating a new ticket that'll implements them in iiif_print.

@ShanaLMoore ShanaLMoore changed the title add derived_members and derived_files override to app/presenters/work_show_presenter Spike: add derived_members and derived_files override to app/presenters/work_show_presenter Jan 6, 2023
@ShanaLMoore ShanaLMoore self-assigned this Jan 9, 2023
@ShanaLMoore
Copy link
Contributor

LV pulled this in from NNP. And yes, it is related to is_derived_ssi.

commit: "decide to show bookreader or not"

@ShanaLMoore
Copy link
Contributor

nnp BookReader: https://github.com/internetarchive/bookreader

@ShanaLMoore
Copy link
Contributor

ShanaLMoore commented Jan 9, 2023

Image

Image

Image

@ShanaLMoore
Copy link
Contributor

Thanks for Kirk for pointing this out. If we bring this info iiif_print, then we may at least need the work related to is_derived for solr.

Todo:

create a ticket to port over work related to setting is_derived in solr.

@ShanaLMoore
Copy link
Contributor

Context:

NNP and LV has a process to define certain files sets are being derivatives of other file
sets.

This needs to decided after LaRita's work is done. We probably won't have to bring over the is_derived however the implementation will be determined after LaRita's work. #11

@ShanaLMoore
Copy link
Contributor

created ticket #297

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Status: Done
Development

No branches or pull requests

3 participants