tulibraries / tul_ohist

Oral Histories Hydra Head for TULibraries
Other
1 stars 1 forks source link

Add collection context box #140

Closed skng5 closed 8 years ago

skng5 commented 8 years ago

Ref #40

Size and crop collection thumbnail

Clean up logic and html

bibliotechy commented 8 years ago

@skng5 Feinstein and Massey Philli[ps records work fine, but the example audio records you provided are throwing errors:

Showing /home/chad/projects/tul_ohist/app/views/catalog/_show_transcript.html.erb where line #69 raised:

undefined method `[]' for nil:NilClass

Extracted source (around line #69):

   </div>
   </div>
   <div class="right-column">
     <%= render_related_resources(document['master_identifier_ssim'].to_sentence, document) %>
   </div>
   <script type='text/javascript'>
       function embedPDF(){

Trace of template inclusion: app/views/catalog/show.html.erb

Rails.root: /home/chad/projects/tul_ohist Application Trace | Framework Trace | Full Trace

app/helpers/tul_ohist_helper.rb:72:in render_related_resources' app/views/catalog/_show_transcript.html.erb:69:in_app_views_catalog__show_transcript_html_erb2735778848331708460_70018415150980' app/views/catalog/show.html.erb:9:in `_app_views_catalog_show_htmlerb4556138881996867540_70018415466920'

Request

Parameters:

{"id"=>"transcript:AOHWMPJZ2014110008xp16002coll22x245"}

Could this just be related to the example records you provided?

bibliotechy commented 8 years ago

I guess a follow up question is - do we have real data now?

skng5 commented 8 years ago

No real data yet. That which we have is test data with links to pre-existing audio which have no association with the digital objects.

skng5 commented 8 years ago

@bibliotechy - I'll look into the error.

bibliotechy commented 8 years ago

Superseded by #144