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

[Port dspace-7_x] Fix for User profile (/profile): only 20 group memberships shown instead of all #3826

Merged
merged 1 commit into from
Jan 10, 2025

Conversation

tdonohue
Copy link
Member

Manual port of #3105 by @VictorHugoDuranS to dspace-7_x

…ead of all (DSpace#3105)

* Add - groups paginated on profile page

* Add - groups paginated on profile page

* Add - groups paginated on profile page

* Add - groups paginated on profile page

* Fix UPDATE - Add pagination message error and loader

* Update BRANCH

* Fix - LINT ERRORS

* Fix: Error declaring variable for group pagination

* Fix: Remove unnecessary translations for paging groups

* Fix: Lint erros

* Fix: Remove unnecessary translations

---------

Co-authored-by: VictorDuranEscire <victor@escire.lat>
@tdonohue tdonohue added bug component: user profile 1 APPROVAL pull request only requires a single approval to merge labels Jan 10, 2025
@tdonohue tdonohue added this to the 7.6.3 milestone Jan 10, 2025
@tdonohue
Copy link
Member Author

👍 I had to update this port slightly because of merge conflicts & the fact that 7.x doesn't have standalone components. But, I've tested it manually & it works!

@tdonohue tdonohue merged commit 6f5864d into DSpace:dspace-7_x Jan 10, 2025
12 of 13 checks passed
@tdonohue tdonohue deleted the port_3105_to_7x branch January 10, 2025 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 APPROVAL pull request only requires a single approval to merge bug component: user profile
Projects
Development

Successfully merging this pull request may close these issues.

2 participants