Bug #11281

Fix issue with genre access point EAD importing

Added by Mike Cantelon about 3 years ago. Updated almost 3 years ago.

Status:VerifiedStart date:06/22/2017
Priority:MediumDue date:
Assignee:-% Done:

0%

Category:EAD
Target version:Release 2.4.0
Google Code Legacy ID: Tested version:2.4
Sponsored:No Requires documentation:

Description

Currently doesn't allow importing of anything with a source attribute: https://github.com/artefactual/atom/blob/qa/2.4.x/apps/qubit/modules/object/config/import/ead.yml#L395

Change so it's something like Dave J's recommendation: "(controlaccess/genreform[not(@source='RAD') and not(@encodinganalog='1.1C')] | controlaccess/controlaccess/genreform[not(@source='RAD') and not(@encodinganalog='1.1C')])".

History

#1 Updated by David Juhasz about 3 years ago

  • Target version set to Release 2.4.0

#2 Updated by David Juhasz about 3 years ago

  • Tested version 2.4 added

#4 Updated by Nick Wilkinson about 3 years ago

  • Assignee changed from Mike Cantelon to José Raddaoui Marín

Hey Radda -- re-assigning this to you to balance out workloads.

#5 Updated by José Raddaoui Marín about 3 years ago

  • Status changed from New to Code Review
  • Assignee changed from José Raddaoui Marín to Nick Wilkinson

I have changed the EAD import and I found and fixed an issue with the directly related IOs facet in the genre index page.

PR 581

#6 Updated by Nick Wilkinson about 3 years ago

  • Assignee changed from Nick Wilkinson to Mike Gale

Hi Mike, assigning to you for CR.

#7 Updated by Mike Gale about 3 years ago

  • Assignee changed from Mike Gale to José Raddaoui Marín

looks good

#8 Updated by José Raddaoui Marín about 3 years ago

  • Status changed from Code Review to QA/Review
  • Assignee changed from José Raddaoui Marín to Nick Wilkinson

Merged in qa/2.4.x

#9 Updated by Nick Wilkinson about 3 years ago

  • Assignee deleted (Nick Wilkinson)

#10 Updated by Dan Gillean almost 3 years ago

  • Status changed from QA/Review to Verified

Also available in: Atom PDF