Skip to content

Commit

Permalink
CAL-929 Add metadata field for "Illustrator" (#831)
Browse files Browse the repository at this point in the history
* CAL-929
Add metadata field for "Illustrator"

Connected to[CAL-929](https://jira.library.ucla.edu/browse/CAL-929)

---

1. `app/controllers/catalog_controller.rb`
1. `app/forms/hyrax/californica_collections_form.rb`
1. `app/forms/hyrax/work_form.rb`
1. `app/importers/californica_mapper.rb`
1. `app/models/solr_document.rb`
1. `app/models/ucla_metadata.rb`
1. `app/presenters/hyrax/californica_collection_presenter.rb`
1. `app/presenters/hyrax/work_presenter.rb`
1. `app/views/hyrax/base/_attribute_rows.html.erb`
1. `solr/config/solrconfig.xml`
1. `spec/fixtures/coordinates_example.csv`
1. `spec/fixtures/coordinates_example_update.csv`
1. `spec/fixtures/csv_import/good/all_fields.csv`
1. `spec/fixtures/example.csv`
1. `spec/fixtures/simple_example.csv`
1. `spec/forms/hyrax/work_form_spec.rb`
1. `spec/importers/californica_csv_parser_spec.rb`
1. `spec/importers/californica_mapper_spec.rb`
1. `spec/system/edit_collection_spec.rb`
1. `spec/system/edit_work_spec.rb`
1. `spec/system/import_and_show_work_spec.rb`
1. `spec/system/show_collection_spec.rb`
1. `spec/uploaders/csv_manifest_validator_spec.rb`
1. `spec/views/hyrax/base/_attribute_rows.html.erb_spec.rb`

* Fix edit_work_spec

* Update coordinates_example csv

* Update expectation import_and_show_work_spec

Co-authored-by: parinita <[email protected]>
  • Loading branch information
jendiamond and pghorpade authored Aug 6, 2020
1 parent d7db09f commit 4c57e71
Show file tree
Hide file tree
Showing 25 changed files with 59 additions and 16 deletions.
2 changes: 1 addition & 1 deletion app/assets/csv/import_manifest.csv
Original file line number Diff line number Diff line change
@@ -1 +1 @@
Object Type,Title,Item ARK,Parent ARK,Rights.copyrightStatus,File Name,AltIdentifier.local,AltTitle.other,Binding note,Condition note,Coverage.geographic,Date.creation,Date.normalized,Description.caption,Description.fundingNote,Description.latitude,Description.longitude,Description.note,Format.dimensions,Format.extent,Format.medium,Language,Name.architect,Name.photographer,Name.repository,Name.subject,Name.repository,Opac url,Project Name,Publisher.publisherName,Relation.isPartOf,Rights.countryCreation,Rights.rightsHolderContact,Rights.statementLocal,Subject,Type.genre,Type.typeOfResource,Place of origin,AltTitle.uniform,Support,Summary,Page layout,Text direction,viewingHint,IIIF Range,Illustrations note,Provenance; Description.history,Table of Contents; Description.tableOfContents,Subject.conceptTopic,Subject.descriptiveTopic,Collation,Foliation note,Foliation,Illuminator,Name.illuminator,Name.lyricist,Name.composer,Scribe,Name.scribe,Masthead,Representative image,Featured image,Tagline,Subject geographic,Subject temporal,Translator,Name.translator,Commentator,Name.commentator,Colophon,Finding Aid URL, Alt ID.url, Rubricator,Name.rubricator,License
Object Type,Title,Item ARK,Parent ARK,Rights.copyrightStatus,File Name,AltIdentifier.local,AltTitle.other,Binding note,Condition note,Coverage.geographic,Date.creation,Date.normalized,Description.caption,Description.fundingNote,Description.latitude,Description.longitude,Description.note,Format.dimensions,Format.extent,Format.medium,Language,Name.architect,Name.photographer,Name.repository,Name.subject,Name.repository,Opac url,Project Name,Publisher.publisherName,Relation.isPartOf,Rights.countryCreation,Rights.rightsHolderContact,Rights.statementLocal,Subject,Type.genre,Type.typeOfResource,Place of origin,AltTitle.uniform,Support,Summary,Page layout,Text direction,viewingHint,IIIF Range,Illustrations note,Provenance; Description.history,Table of Contents; Description.tableOfContents,Subject.conceptTopic,Subject.descriptiveTopic,Collation,Foliation note,Foliation,Illuminator,Name.illuminator,Name.lyricist,Name.composer,Scribe,Name.scribe,Masthead,Representative image,Featured image,Tagline,Subject geographic,Subject temporal,Translator,Name.translator,Commentator,Name.commentator,Colophon,Finding Aid URL, Alt ID.url, Rubricator,Name.rubricator,License,Illustrator,Name.illustrator
5 changes: 5 additions & 0 deletions app/assets/markdown/importer_guide.md
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,7 @@
- [IIIF Access URL](#iiif-access-url)
- [IIIF Range](#iiif-range)
- [Illustrations note](#illustrations-note)
- [Illustrator](#illustrator)
- [Item Sequence](#item-sequence)
- [Language](#language)
- [License](#license)
Expand Down Expand Up @@ -234,6 +235,10 @@ Also aaccepts: `Name.illuminator`

Also accepts: `Description.illustrations`

### Illustrator

accepts ""Illustrator", "Name.illustrator"

### Item Sequence

### viewingHint
Expand Down
10 changes: 9 additions & 1 deletion app/controllers/catalog_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,6 @@ def self.modified_field
config.add_show_field 'format_tesim'
config.add_show_field 'identifier_tesim'
config.add_show_field 'ark_ssi', label: 'ARK'

config.add_show_field 'access_copy_ssi'
config.add_show_field 'alternative_title_tesim'
config.add_show_field 'architect_tesim'
Expand All @@ -125,6 +124,7 @@ def self.modified_field
config.add_show_field 'iiif_viewing_hint_ssi'
config.add_show_field 'illuminator_tesim'
config.add_show_field 'illustrations_note_tesim'
config.add_show_field 'illustrator_tesim'
config.add_show_field 'location_tesim'
config.add_show_field 'local_identifier_ssm'
config.add_show_field 'lyricist_tesim'
Expand Down Expand Up @@ -419,6 +419,14 @@ def self.modified_field
}
end

config.add_search_field('illustrator') do |field|
solr_name = solr_name('illustrator', :stored_searchable)
field.solr_local_parameters = {
qf: solr_name,
pf: solr_name
}
end

config.add_search_field('keyword') do |field|
solr_name = solr_name('keyword', :stored_searchable)
field.solr_local_parameters = {
Expand Down
1 change: 1 addition & 0 deletions app/forms/hyrax/californica_collections_form.rb
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ class CalifornicaCollectionsForm < Hyrax::Forms::CollectionForm
:iiif_manifest_url,
:iiif_viewing_hint,
:illuminator,
:illustrator,
:illustrations_note,
:latitude,
:longitude,
Expand Down
1 change: 1 addition & 0 deletions app/forms/hyrax/work_form.rb
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ class WorkForm < Hyrax::Forms::WorkForm
:iiif_viewing_hint,
:illuminator,
:illustrations_note,
:illustrator,
:latitude,
:license,
:local_identifier,
Expand Down
4 changes: 3 additions & 1 deletion app/importers/californica_mapper.rb
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,9 @@ class CalifornicaMapper < Darlingtonia::HashMapper
iiif_manifest_url: "IIIF Manifest URL",
iiif_range: "IIIF Range",
iiif_viewing_hint: "viewingHint",
illustrations_note: ["Illustrations note", "Description.illustrations"],
illuminator: ["Illuminator", "Name.illuminator"],
illustrations_note: ["Illustrations note", "Description.illustrations"],
illustrator: ["Illustrator", "Name.illustrator"],
language: "Language",
latitude: "Description.latitude",
license: "License",
Expand Down Expand Up @@ -173,6 +174,7 @@ def ark
Ark.ensure_prefix(map_field(:ark).to_a.first)
end

# Only single value field need to be defined
def binding_note
map_field(:binding_note).to_a.first
end
Expand Down
4 changes: 4 additions & 0 deletions app/models/solr_document.rb
Original file line number Diff line number Diff line change
Expand Up @@ -126,6 +126,10 @@ def illustrations_note
self[:illustrations_note_tesim]
end

def illustrator
self[:illustrator_tesim]
end

def latitude
self[Solrizer.solr_name('latitude')]
end
Expand Down
4 changes: 4 additions & 0 deletions app/models/ucla_metadata.rb
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,10 @@ module UclaMetadata
index.as :stored_searchable
end

property :illustrator, predicate: ::RDF::URI.intern('http://id.loc.gov/vocabulary/relators/ill.html') do |index|
index.as :stored_searchable
end

property :latitude, predicate: ::RDF::Vocab::EXIF.gpsLatitude do |index|
index.as :stored_searchable
end
Expand Down
1 change: 1 addition & 0 deletions app/presenters/hyrax/californica_collection_presenter.rb
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ def self.terms
:iiif_manifest_url,
:iiif_range,
:illustrations_note,
:illustrator,
:iiif_viewing_hint,
:identifier,
:illuminator,
Expand Down
1 change: 1 addition & 0 deletions app/presenters/hyrax/work_presenter.rb
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ class WorkPresenter < Hyrax::WorkShowPresenter
:iiif_manifest_url,
:iiif_range,
:illustrations_note,
:illustrator,
:iiif_viewing_hint,
:illuminator,
:license,
Expand Down
1 change: 1 addition & 0 deletions app/views/hyrax/base/_attribute_rows.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@
<%= presenter.attribute_to_html(:iiif_viewing_hint, render_as: :iiif_viewing_hint, html_dl: true) %>
<%= presenter.attribute_to_html(:illuminator, html_dl: true) %>
<%= presenter.attribute_to_html(:illustrations_note, html_dl: true) %>
<%= presenter.attribute_to_html(:illustrator, html_dl: true) %>
<%= presenter.attribute_to_html(:keyword, render_as: :faceted, html_dl: true) %>
<%= presenter.attribute_to_html(:language, render_as: :language, html_dl: true) %>
<%= presenter.attribute_to_html(:local_identifier, html_dl: true) %>
Expand Down
4 changes: 2 additions & 2 deletions spec/fixtures/coordinates_example.csv
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
Project Name,Object Type,Parent ARK,Item ARK,Item Status,Subject,Type.typeOfResource,Rights.copyrightStatus,Type.genre,Name.subject,Coverage.geographic,Relation.isPartOf,Publisher.publisherName,Rights.countryCreation,Rights.rightsHolderContact,Name.architect,Name.photographer,Name.repository,Date.normalized,AltIdentifier.local,Title,Date.creation,Format.extent,Format.medium,Format.dimensions,Description.note,Description.fundingNote,Description.longitude,Description.latitude,Description.caption,File Name,AltTitle.other,AltTitle.translated,Place of origin,AltTitle.uniform,Support,Author,Summary,Page layout,Text direction,Binding note,viewingHint,IIIF Range,Illustrations note,Description.history,Table of Contents,Subject.conceptTopic,Subject.descriptiveTopic,Collation,Foliation note, Foliation,Illuminator,Name.illuminator,Name.lyricist,Name.composer,Scribe,Name.scribe,Condition note,Rights.statement,Masthead,Representative image,Featured image,Tagline,Commentator,Name.commentator,Translator,Name.translator,Subject temporal,Opac url,Subject geographic,Colophon,Description.colophon,Finding Aid URL,Alt ID.url,Rubricator,Name.rubricator,Name.creator
Los Angeles Daily News Negatives,Work,111/222,13030/hb338nb26f,Completed with minimal metadata,"Altars|~|Mass|~|Communion table|~|Mexican American Catholics|~|Festivals|~|Guadalupe, Our Lady of|~|Historic buildings $z California $z Los Angeles|~|Churches $z California $z Los Angeles|~|Religion|~|Catholic churches $z California $z Los Angeles ",still image,copyrighted,news photographs,"Plaza Church (Los Angeles, Calif.)",Los Angeles (Calif.)|~|Los Angeles (Calif.),"Los Angeles Daily News Negatives. Department of Special Collections, Charles E. Young Research Library, University of California at Los Angeles.",Los Angeles Daily News,US,"UCLA Charles E. Young Research Library Department of Special Collections, A1713 Young Research Library, Box 951575, Los Angeles, CA 90095-1575. E-mail: [email protected]. Phone: (310)825-4988",Imhotep,Famous Photographer,,1942/1952,uclamss_1387_b112_40911-1,"Communion at Plaza Church, Los Angeles, 1942-1952",[between 1942-1947],Fake Extent,Fake Medium,200x200,"A woman takes Communion during a Plaza Church mass for the fiesta of Nuestra Senora de Guadalupe.|~|Our Lady Queen of Los Angeles Catholic Church, (aka La Iglesia de Nuestra Señora la Reina de Los Ángeles, and The Church of Our Lady the Queen of the Angels), was built in 1822 and is the oldest church in Los Angeles.",Fake Funding Note,-118.239577,34.05707,Fake Caption,clusc_1_1_00010432a.tif,Communion table,Mesa de Comunión,Los Angeles,Mexican American Catholics,UCLA,Famous Author,Mexican American Catholics,images,left-to-right,Green,paged,iiif-range,illustration-note,history-description,table of contents,concept-topic,descriptive-topic,collated,follated,also follated,illuminated,also illuminated,la la la,Schubert,Amat-Mamu,Sin-liqe-unninni,condition_note,local_statement,local_masthead,local_representative_image,local_featured_image,local_tagline,commentator_1,commentator_2,translator_1,translator_2,subject_temporal_1,https://www.library.ucla.edu,subject_geographic_1,colophon_text,colophon_text_2,finding_aid_url_1,finding_aid_url_2,rubricator_1,rubricator_2,name_creator
Project Name,Object Type,Parent ARK,Item ARK,Item Status,Subject,Type.typeOfResource,Rights.copyrightStatus,Type.genre,Name.subject,Coverage.geographic,Relation.isPartOf,Publisher.publisherName,Rights.countryCreation,Rights.rightsHolderContact,Name.architect,Name.photographer,Name.repository,Date.normalized,AltIdentifier.local,Title,Date.creation,Format.extent,Format.medium,Format.dimensions,Description.note,Description.fundingNote,Description.longitude,Description.latitude,Description.caption,File Name,AltTitle.other,AltTitle.translated,Place of origin,AltTitle.uniform,Support,Author,Summary,Page layout,Text direction,Binding note,viewingHint,IIIF Range,Illustrations note,Illustrator,Name.illustrator,Description.history,Table of Contents,Subject.conceptTopic,Subject.descriptiveTopic,Collation,Foliation note, Foliation,Illuminator,Name.illuminator,Name.lyricist,Name.composer,Scribe,Name.scribe,Condition note,Rights.statement,Masthead,Representative image,Featured image,Tagline,Commentator,Name.commentator,Translator,Name.translator,Subject temporal,Opac url,Subject geographic,Colophon,Description.colophon,Finding Aid URL,Alt ID.url,Rubricator,Name.rubricator,Name.creator
Los Angeles Daily News Negatives,Work,111/222,13030/hb338nb26f,Completed with minimal metadata,"Altars|~|Mass|~|Communion table|~|Mexican American Catholics|~|Festivals|~|Guadalupe, Our Lady of|~|Historic buildings $z California $z Los Angeles|~|Churches $z California $z Los Angeles|~|Religion|~|Catholic churches $z California $z Los Angeles ",still image,copyrighted,news photographs,"Plaza Church (Los Angeles, Calif.)",Los Angeles (Calif.)|~|Los Angeles (Calif.),"Los Angeles Daily News Negatives. Department of Special Collections, Charles E. Young Research Library, University of California at Los Angeles.",Los Angeles Daily News,US,"UCLA Charles E. Young Research Library Department of Special Collections, A1713 Young Research Library, Box 951575, Los Angeles, CA 90095-1575. E-mail: [email protected]. Phone: (310)825-4988",Imhotep,Famous Photographer,,1942/1952,uclamss_1387_b112_40911-1,"Communion at Plaza Church, Los Angeles, 1942-1952",[between 1942-1947],Fake Extent,Fake Medium,200x200,"A woman takes Communion during a Plaza Church mass for the fiesta of Nuestra Senora de Guadalupe.|~|Our Lady Queen of Los Angeles Catholic Church, (aka La Iglesia de Nuestra Señora la Reina de Los Ángeles, and The Church of Our Lady the Queen of the Angels), was built in 1822 and is the oldest church in Los Angeles.",Fake Funding Note,-118.239577,34.05707,Fake Caption,clusc_1_1_00010432a.tif,Communion table,Mesa de Comunión,Los Angeles,Mexican American Catholics,UCLA,Famous Author,Mexican American Catholics,images,left-to-right,Green,paged,iiif-range,illustration-note,Illustrator-Person1,Illustrator-PersonName,history-description,table of contents,concept-topic,descriptive-topic,collated,follated,also follated,illuminated,also illuminated,la la la,Schubert,Amat-Mamu,Sin-liqe-unninni,condition_note,local_statement,local_masthead,local_representative_image,local_featured_image,local_tagline,commentator_1,commentator_2,translator_1,translator_2,subject_temporal_1,https://www.library.ucla.edu,subject_geographic_1,colophon_text,colophon_text_2,finding_aid_url_1,finding_aid_url_2,rubricator_1,rubricator_2,name_creator
4 changes: 2 additions & 2 deletions spec/fixtures/coordinates_example_update.csv
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
Project Name,Object Type,Parent ARK,Item ARK,Subject,Type.typeOfResource,Rights.copyrightStatus,Type.genre,Name.subject,Coverage.geographic,Relation.isPartOf,Publisher.publisherName,Rights.countryCreation,Rights.rightsHolderContact,Name.photographer,Name.repository,Date.normalized,AltIdentifier.local,Title,Date.creation,Format.extent,Format.medium,Format.dimensions,Description.note,Description.fundingNote,Description.longitude,Description.latitude,Description.caption,File Name,Author,Page layout,Text direction,Binding note,viewingHint,IIIF Range,Illustrations note,Provenance; Description.history,Table of Contents; Description.tableOfContents,Subject.conceptTopic,Subject.descriptiveTopic,Collation,Foliation note; Foliation,Illuminator,Name.illuminator,Name.lyricist,Name.composer,Scribe,Name.scribe,Condition note,Rights.statementLocal,Masthead,Representative image,Featured image,Tagline,Commentator,Name.commentator,Translator,Name.translator,Subject temporal,Opac url,Subject geographic,Colophon,Description.colophon,Finding Aid URL,Alt ID.url,Rubricator,Name.rubricator,Name.creator
Los Angeles Daily News Negatives,Work,111/222,13030/hb338nb26f,"Altars|~|Mass|~|Communion table|~|Mexican American Catholics|~|Festivals|~|Guadalupe, Our Lady of|~|Historic buildings $z California $z Los Angeles|~|Churches $z California $z Los Angeles|~|Religion|~|Catholic churches $z California $z Los Angeles ",still image,copyrighted,news photographs,"Plaza Church (Los Angeles, Calif.)",Los Angeles (Calif.)|~|Los Angeles (Calif.),"Los Angeles Daily News Negatives. Department of Special Collections, Charles E. Young Research Library, University of California at Los Angeles.",Los Angeles Daily News,US,"UCLA Charles E. Young Research Library Department of Special Collections, A1713 Young Research Library, Box 951575, Los Angeles, CA 90095-1575. E-mail: [email protected]. Phone: (310)825-4988",Famous Photographer,,1942/1952,uclamss_1387_b112_40911-1,"Communion at Plaza Church, Los Angeles, 1942-1952",[between 1942-1947],Better Extent,,200x200,"A woman takes Communion during a Plaza Church mass for the fiesta of Nuestra Senora de Guadalupe.|~|Our Lady Queen of Los Angeles Catholic Church, (aka La Iglesia de Nuestra Señora la Reina de Los Ángeles, and The Church of Our Lady the Queen of the Angels), was built in 1822 and is the oldest church in Los Angeles.",Better Funding Note,-118.239577,34.05707,Better Caption,clusc_1_1_00010432a.tif, Famous Author,images,left-to-right,Green,paged,iiif-range,illustration-note,history-description,table of contents,concept-topic,descriptive-topic,collated,follated,illuminated,also illuminated,la la la,Schubert,Amat-Mamu,Sin-liqe-unninni,condition_note,local_statement,local_masthead,local_representative_image,local_featured_image,local_tagline,commentator_1,commentator_2,translator_1,translator_2,subject_temporal_1,https://www.linrary.ucla.edu,subject_geographic_1,colophon_text,colophon_text_2,finding_aid_url_1,finding_aid_url_2,rubricator_1,rubricator_2,name_creator
Project Name,Object Type,Parent ARK,Item ARK,Subject,Type.typeOfResource,Rights.copyrightStatus,Type.genre,Name.subject,Coverage.geographic,Relation.isPartOf,Publisher.publisherName,Rights.countryCreation,Rights.rightsHolderContact,Name.photographer,Name.repository,Date.normalized,AltIdentifier.local,Title,Date.creation,Format.extent,Format.medium,Format.dimensions,Description.note,Description.fundingNote,Description.longitude,Description.latitude,Description.caption,File Name,Author,Page layout,Text direction,Binding note,viewingHint,IIIF Range,Illustrations note,Provenance; Description.history,Table of Contents; Description.tableOfContents,Subject.conceptTopic,Subject.descriptiveTopic,Collation,Foliation note; Foliation,Illuminator,Name.illuminator,Name.lyricist,Name.composer,Scribe,Name.scribe,Condition note,Rights.statementLocal,Masthead,Representative image,Featured image,Tagline,Commentator,Name.commentator,Translator,Name.translator,Subject temporal,Opac url,Subject geographic,Colophon,Description.colophon,Finding Aid URL,Alt ID.url,Rubricator,Name.rubricator,Name.creator,Illustrator,Name.illustrator
Los Angeles Daily News Negatives,Work,111/222,13030/hb338nb26f,"Altars|~|Mass|~|Communion table|~|Mexican American Catholics|~|Festivals|~|Guadalupe, Our Lady of|~|Historic buildings $z California $z Los Angeles|~|Churches $z California $z Los Angeles|~|Religion|~|Catholic churches $z California $z Los Angeles ",still image,copyrighted,news photographs,"Plaza Church (Los Angeles, Calif.)",Los Angeles (Calif.)|~|Los Angeles (Calif.),"Los Angeles Daily News Negatives. Department of Special Collections, Charles E. Young Research Library, University of California at Los Angeles.",Los Angeles Daily News,US,"UCLA Charles E. Young Research Library Department of Special Collections, A1713 Young Research Library, Box 951575, Los Angeles, CA 90095-1575. E-mail: [email protected]. Phone: (310)825-4988",Famous Photographer,,1942/1952,uclamss_1387_b112_40911-1,"Communion at Plaza Church, Los Angeles, 1942-1952",[between 1942-1947],Better Extent,,200x200,"A woman takes Communion during a Plaza Church mass for the fiesta of Nuestra Senora de Guadalupe.|~|Our Lady Queen of Los Angeles Catholic Church, (aka La Iglesia de Nuestra Señora la Reina de Los Ángeles, and The Church of Our Lady the Queen of the Angels), was built in 1822 and is the oldest church in Los Angeles.",Better Funding Note,-118.239577,34.05707,Better Caption,clusc_1_1_00010432a.tif, Famous Author,images,left-to-right,Green,paged,iiif-range,illustration-note,history-description,table of contents,concept-topic,descriptive-topic,collated,follated,illuminated,also illuminated,la la la,Schubert,Amat-Mamu,Sin-liqe-unninni,condition_note,local_statement,local_masthead,local_representative_image,local_featured_image,local_tagline,commentator_1,commentator_2,translator_1,translator_2,subject_temporal_1,https://www.linrary.ucla.edu,subject_geographic_1,colophon_text,colophon_text_2,finding_aid_url_1,finding_aid_url_2,rubricator_1,rubricator_2,name_creator,Illustrator-Person1,Illustrator-PersonName
Loading

0 comments on commit 4c57e71

Please sign in to comment.