-
Notifications
You must be signed in to change notification settings - Fork 17
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 separate Publisher field. #138
Conversation
It occurs to me that Citation Select is a place I did forget, where we could* set this. However, Citation Select only lets us map one field, for all content, to (csl) publisher. Right now it's using Linked Agent but we could switch it over, resulting in nodes that use Contributors (Linked Agent) for publisher not having a publisher in the citation. |
config/sync/field.field.node.islandora_object.field_linked_agent.yml
Outdated
Show resolved
Hide resolved
discussed in the tech call. Decided to go all-in on the Publisher field, and remove relators:pbl from the Contributors field. |
…ch facet and advanced search.
I've gone "all in" on publisher. I removed it from the linked agent options. I also added and placed a facet, both to the main and the collection search views. I also configured it as an option in the "fielded search" dropdown, since it's fulltext. |
This is a great change! e.g. our I haven't full tested this PR (I just made the changes manually), but did write a script to move the publisher linked agents to the new |
Tested by spinning up[ a new ISLE-DC and replacing the Git URL in the make starter_dev command. The field is in the node add form, and there's a helpful note under the contributors widget to add publisher to its own field. JSON-LD has the field as expected: {"@graph":[{"@id":"https:\/\/islandora.traefik.me\/node\/1","@type":["http:\/\/pcdm.org\/models#Object","https:\/\/schema.org\/Book"],"http:\/\/schema.org\/author":[{"@id":"https:\/\/islandora.traefik.me\/user\/1"}],"http:\/\/purl.org\/dc\/terms\/title":[{"@value":"Test object with publisher field","@language":"en"}],"http:\/\/schema.org\/dateCreated":[{"@value":"2024-04-02T22:27:25+00:00","@type":"http:\/\/www.w3.org\/2001\/XMLSchema#dateTime"}],"http:\/\/schema.org\/dateModified":[{"@value":"2024-04-02T22:27:25+00:00","@type":"http:\/\/www.w3.org\/2001\/XMLSchema#dateTime"}],"http:\/\/purl.org\/dc\/elements\/1.1\/publisher":[{"@value":"Words on Paper","@type":"http:\/\/www.w3.org\/2001\/XMLSchema#string"}],"http:\/\/schema.org\/sameAs":[{"@id":"https:\/\/islandora.traefik.me\/node\/1"}]},{"@id":"https:\/\/islandora.traefik.me\/user\/1","@type":["http:\/\/schema.org\/Person"]}]} It's indexed in Solr It shows up as a citation: Test Object With Publisher Field. Words on Paper. Review all citations for accuracy. |
@joecorall this is hot on the heels of another pull request that lets you exclude/include relators from a facet. So double fix! I too wrote a script and made a video about it. I'll link them all from the release notes. |
Issue #137
This PR should create a new field for Publisher, as requested by the metadata Interest Group.
To test: Spin up a fresh site with this set of configs.
Please add a comment if there are places I've forgotten!