Feature #10048

Provide pagination and "show all" link in authority records linked descriptions lists

Added by José Raddaoui Marín about 4 years ago. Updated almost 3 years ago.

Status:VerifiedStart date:06/09/2016
Priority:MediumDue date:
Assignee:-% Done:

0%

Category:Actor
Target version:Release 2.4.0
Google Code Legacy ID: Tested version:
Sponsored:Yes Requires documentation:

Description

This work includes 2 parts:

1) Show all links: add a 'Show all' link to 'Creator of' and 'Subject of' lists (using existing facets), and add a 'Show all' link to other lists (Contributor, Publisher, etc; requires changes to Elasticsearch because no facets or filters are currently set up for these)

2) Add pagination to all lists

example-browse-links.png (74 KB) Dan Gillean, 06/28/2016 07:45 PM

example-pager.png (56.7 KB) Dan Gillean, 06/28/2016 07:45 PM

one-to-twenty-test.csv Magnifier (41 KB) Dan Gillean, 06/28/2016 07:46 PM


Related issues

Related to Access to Memory (AtoM) - Bug #11828: Authority records left side links shows Creator and Subje... Verified 12/28/2017

History

#2 Updated by José Raddaoui Marín about 4 years ago

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

Ready for code review in PR 390

#3 Updated by Nick Wilkinson about 4 years ago

  • Assignee changed from Nick Wilkinson to Mike Gale

Hi Mike, assigning to you for CR.

#4 Updated by Mike Gale about 4 years ago

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

Looks good, a couple minor comments.

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

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

Hi Mike, please, could you take a quick look again? I've added your feedback and a commit to fix an error in those paginations if you're in dev mode. Thanks.

#6 Updated by Mike Gale about 4 years ago

  • Status changed from Code Review to Feedback
  • Assignee changed from Mike Gale to José Raddaoui Marín

one question!

#7 Updated by José Raddaoui Marín about 4 years ago

  • Status changed from Feedback to QA/Review
  • Assignee changed from José Raddaoui Marín to Dan Gillean

Merged in qa/2.4.x

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

  • Requires documentation set to Yes

About documentation, a side effect of this feature is that the new parameters used for some of the browse all links: 'eventTypeId' and 'actorId', are available in the CSV export in the IO browse page and in the REST API IO browse endpoint. They are used to filter IOs by their event actors and they only work when they are sent together.

#9 Updated by Dan Gillean about 4 years ago

Sorry Radda, I don't totally understand what you mean by that - could you provide an example perhaps?

#10 Updated by Dan Gillean about 4 years ago

Looks good. Attaching a couple screenshots for public reference. Also attaching a test CSV that will create 20 near-identical descriptions, with titles running from "one" to "twenty", all linked to the same creator and name access points.

The one improvement I would say we should consider would be suppressing the browse results link if there is only 1 result - this will help keep the UI free of unnecessary visual clutter for a common use case.

Marking verified for now. If there is any time remaining on the related client ticket, maybe we can consider the above suggestion.

#11 Updated by Dan Gillean almost 3 years ago

  • Assignee deleted (Dan Gillean)
  • Requires documentation deleted (Yes)

#12 Updated by Dan Gillean over 2 years ago

  • Related to Bug #11828: Authority records left side links shows Creator and Subject when only Creator is expected added

Also available in: Atom PDF