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.

Sign up for GitHub

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

Jump to bottom

DS-4342 improve performance when retrieving items from a collection (7.x) #2730

Merged
merged 1 commit into from
Jun 29, 2020
Merged

DS-4342 improve performance when retrieving items from a collection (7.x) #2730

merged 1 commit into from
Jun 29, 2020

Conversation

nwoodward
Copy link
Contributor

Port of #2516

tdonohue
tdonohue approved these changes Jun 29, 2020
View reviewed changes
Copy link
Member

@tdonohue tdonohue left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Looks good. Merging this immediately as it's a forward-port of #2516 and it only impacts the deprecated v4-6 REST API.

@tdonohue tdonohue added bug interface: REST API v4-6 (obsolete) Old REST API in v4.x, 5.x and 6.x, deprecated in 7.x. Removed in 8.x (dspace-rest module) quick win Pull request is small in size & should be easy to review and/or merge labels Jun 29, 2020
@tdonohue tdonohue merged commit 1e14ebb into DSpace:master Jun 29, 2020
@tdonohue tdonohue added this to the 7.0beta3 milestone Jan 26, 2021
@nwoodward nwoodward deleted the DS-4342-7x branch March 6, 2023 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tdonohue tdonohue tdonohue approved these changes

Assignees
No one assigned
Labels
bug interface: REST API v4-6 (obsolete) Old REST API in v4.x, 5.x and 6.x, deprecated in 7.x. Removed in 8.x (dspace-rest module) quick win Pull request is small in size & should be easy to review and/or merge
Projects
None yet
Milestone
7.0beta3
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@nwoodward @tdonohue