Skip to content
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

The associations to AIP panel is highly incomplete #1083

Closed
jmaferreira opened this issue Nov 22, 2017 · 9 comments
Closed

The associations to AIP panel is highly incomplete #1083

jmaferreira opened this issue Nov 22, 2017 · 9 comments
Assignees
Labels
Milestone

Comments

@jmaferreira
Copy link
Member

jmaferreira commented Nov 22, 2017

  • Design is broken - labels should align left (it should follow the design of the advanced search)
  • Textboxes should be white
  • Window is too big
  • It should contain more attributes to search on (look at the advanced search options)

screen shot 2017-11-22 at 18 02 06

@jmaferreira jmaferreira added this to the 2.2.0 milestone Nov 22, 2017
@chalkos
Copy link
Contributor

chalkos commented Nov 29, 2017

  • Design is broken - labels should align left (it should follow the design of the advanced search)
  • Textboxes should be white -- same as Text boxes should be white on RI panel and combos blue #1084, fixed
  • Window is too big
  • It should contain more attributes to search on (look at the advanced search options) -- changed via configuration parameters

@chalkos
Copy link
Contributor

chalkos commented Nov 29, 2017

@jmaferreira about the last point, I'm looking at the adv search but it doesnt make sense to put those options here. eg: you can search for intellectual entities' description but it makes no sense to add a relation to intellectual entities that have a certain description. Maybe you were thinking that this is a search form... this is where you specify the relation rules (eg: all MIXED AIP should be related to this RI).

chalkos added a commit that referenced this issue Nov 29, 2017
@jmaferreira
Copy link
Member Author

That is correct. However, there are more attributes that are useful here. e.g. AIP ID.
I need a list of all of them to pick the ones that matter.

@chalkos
Copy link
Contributor

chalkos commented Dec 5, 2017

I looked up the solr fields, removed some fields like updatedOn and multivalued fields, and ended up with this list:

AIP

  • id
  • type
  • createdBy
  • updatedBy
  • level
  • title
  • ghost
  • state

File

  • fileId
  • formatMimetype
  • formatVersion
  • formatPronom
  • fileFormat
  • formatDesignation
  • extension
  • creatingApplicationName
  • creatingApplicationVersion
  • state

Representation

  • id
  • original
  • type
  • createdBy
  • updatedBy
  • state

@luis100 may be able to help you understand where each field is used.

@jmaferreira
Copy link
Member Author

Let's go with these, please:

AIP

  • id
  • type
  • level
  • state?

File

  • fileId
  • formatMimetype
  • formatVersion
  • formatPronom
  • fileFormat
  • formatDesignation
  • extension
  • creatingApplicationName
  • creatingApplicationVersion
  • state?

Representation

  • id
  • original
  • type
  • state?

PS: I'm not too sure if we should include the "state". @luis100, what's your take on this?

@luis100
Copy link
Member

luis100 commented Dec 5, 2017

The list you have received is not complete because multivalue fields have been removed. For example, Representation states (to be renamed in the future) has original, preservation and other status of the representation.

The state here refers to the AIPState, being it CREATED, INGEST_PROCESSING, UNDER_APPRAISAL, ACTIVE, DELETED. Only ACTIVE AIPs are shown on catalogue, so it makes no sense to put it here as a filter.

Note that descriptive metadata will make it necessary to add AIP dynamic fields like *_txt

@chalkos
Copy link
Contributor

chalkos commented Dec 5, 2017

Here are the multivalue fields:

AIP

  • ancestors
  • descriptiveMetadataId
  • representationId
  • ingestSIPIds
  • ingestUpdateJobIds
  • description

File

  • path
  • hash
  • ancestors
  • ancestorsPath
  • ingestSIPIds
  • ingestUpdateJobIds

Representation

  • ancestors
  • representationStates
  • ingestSIPIds
  • ingestUpdateJobIds

I removed the size and count fields, since it makes no sense to associate those with RI. I included the id fields, but I think that the only field from this list that makes sense to have associated RI is representationStates

chalkos added a commit that referenced this issue Jan 3, 2018
chalkos added a commit that referenced this issue Jan 3, 2018
@chalkos
Copy link
Contributor

chalkos commented Jan 4, 2018

as it seems, changing the fields used to associate RI with AIP/Representation/File is not simply changing a configuration:

  • the red and blue (i) icons need to be added to the (new) fields used by representation information
  • some of the fields are hidden or in a place where the (i) icon does not visually fit
  • some of the fields would be better shown as a dropdown (similar to how they are shown in the advanced search, which may require a rewrite of a lot of code)

Me and @nunovieira220 will try to find a good solution for these topics.

the other problems described in this issue have been fixed.

@nunovieira220
Copy link
Contributor

Done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants