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

Collection service backport #2040

Merged
merged 2 commits into from
Nov 27, 2023
Merged

Collection service backport #2040

merged 2 commits into from
Nov 27, 2023

Conversation

jeremyf
Copy link
Contributor

@jeremyf jeremyf commented Nov 27, 2023

🐛 Fix Add to Collection for page 2+ of works

b4e2d6a

Prior to this commit, when you were on page 2 of your works and selected
a work to add to a collection, the query for available collections would
use the page 2 as part of the collection query. This would mean the
first 100 collections (default page size) that you had access to add
works to were skipped.

With this commit, we omit the query parameters from the works page and
then query collections.

Related to:

Co-authored-by: LaRita Robinson larita@scientist.com

🐛 Fix bad method name

e201df0

Related to:

jeremyf and others added 2 commits November 27, 2023 16:35
Prior to this commit, when you were on page 2 of your works and selected
a work to add to a collection, the query for available collections would
use the page 2 as part of the collection query.  This would mean the
first 100 collections (default page size) that you had access to add
works to were skipped.

With this commit, we omit the query parameters from the works page and
then query collections.

Related to:

- samvera/hyrax#5972
- samvera/hyrax#5969
- https://github.com/scientist-softserv/adventist-dl/issues/625

Co-authored-by: LaRita Robinson <larita@scientist.com>
@jeremyf jeremyf added the patch-ver for release notes label Nov 27, 2023
@jeremyf jeremyf merged commit 2f36cee into main Nov 27, 2023
4 of 5 checks passed
@jeremyf jeremyf deleted the collection-service-backport branch November 27, 2023 22:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch-ver for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants