Skip to content

Remove sorting of users in js, rely on server side - #40840

Merged
jvillafanez merged 3 commits into
masterfrom
remove_client_sorting
Aug 1, 2023
Merged

Remove sorting of users in js, rely on server side#40840
jvillafanez merged 3 commits into
masterfrom
remove_client_sorting

Conversation

@jvillafanez

Copy link
Copy Markdown
Member

Description

Previously, users where sorted in the server alphabetically (by displayname it seems), and then the js code sort them again by username with a natural sort. This is weird because the next bunch of user might not be added at the bottom of the list as it's expected, and it might appear at the beginning of the list.

We'll rely on the server to sort the results, so the client will just need to append the results at the end of the list.

Related Issue

https://github.com/owncloud/enterprise/issues/5634

Motivation and Context

How Has This Been Tested?

Manually tested

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Database schema changes (next release will require increase of minor version instead of patch)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:
  • Changelog item, see TEMPLATE
@sonarqubecloud

Copy link
Copy Markdown
@owncloud owncloud deleted a comment from update-docs Bot Jun 15, 2023
@pako81
pako81 self-requested a review July 31, 2023 11:44
@pako81

pako81 commented Jul 31, 2023

Copy link
Copy Markdown

I guess we should try to get this in the 10.13 release given that owncloud/user_ldap#796 will be included in user_ldap 0.18.0 and that this app version will be bundled in core 10.13. So if we not merge this, anyone using 10.13 + user_ldap 0.18.0 will notice inconsistent behaviour between LDAP groups and all users sort order.

@pako81

pako81 commented Aug 1, 2023

Copy link
Copy Markdown

@jvillafanez merge this or are tests still needed?

@jvillafanez
jvillafanez merged commit 8c98a26 into master Aug 1, 2023
@delete-merged-branch
delete-merged-branch Bot deleted the remove_client_sorting branch August 1, 2023 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

2 participants