Skip to content
This repository has been archived by the owner on Oct 15, 2021. It is now read-only.

Display publications on the grants show page #190

Merged
merged 1 commit into from Nov 8, 2018
Merged

Conversation

jcoyne
Copy link
Contributor

@jcoyne jcoyne commented Nov 8, 2018

Fixes #112

@ghost ghost assigned jcoyne Nov 8, 2018
@ghost ghost added the review label Nov 8, 2018
Copy link
Member

@mjgiarlo mjgiarlo left a comment

Choose a reason for hiding this comment

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

Have a question but it's nbd.

@@ -160,12 +160,13 @@ class CatalogController < ApplicationController
pc.add_show_field 'subject_label_tsim', label: 'Subject'
pc.add_show_field 'link_ssim', label: 'Link'
pc.add_show_field 'publisher_label_tsim', label: 'Publisher'
pc.add_show_field 'Grants', label: 'Grant', accessor: :linked_grants
pc.add_show_field 'grants', label: 'Grant', accessor: :linked_grants
Copy link
Member

Choose a reason for hiding this comment

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

Should the label here be Grants, or are you thinking publications will be associated with only one grant?

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 suspect most will only have one grant...but we'll find out.

Copy link
Member

Choose a reason for hiding this comment

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

Easy change to make later when we know more. 👍

@mjgiarlo mjgiarlo merged commit d4e81a3 into master Nov 8, 2018
@mjgiarlo mjgiarlo deleted the show-publications branch November 8, 2018 23:46
@ghost ghost removed the review label Nov 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants