[feature/sort-order] Localized sort order #1064
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
|
felix-schwarz
changed the base branch from
milestone/11.8
to
milestone/11.7.2
hosy
approved these changes
Nov 30, 2021
|
QA checks:
Approved on my side |
jesmrec
approved these changes
Dec 1, 2021
|
Branch is based on |
felix-schwarz
changed the base branch from
milestone/11.9.0
to
milestone/11.8.1
- make SortMethod.comparator use OCSQLiteCollationLocalized.sortComparator
felix-schwarz
force-pushed
the
feature/sort-order
branch
from
72b07f7 to
4b8b2c6
Compare
felix-schwarz
changed the base branch from
milestone/11.8
to
milestone/11.8.1
hosy
added a commit
that referenced
this pull request
Dec 22, 2021
* preparing bug fix release 11.8.1 * - added in-app release notes - added fastlane changelog - added Calens changelog entry * Calens changelog updated * [feature/oidc-dcr-fallback] Fallback on/ignore OIDC Dynamic Client Registration errors (#1068) * - update SDK to add support for authentication-oauth2.oidc-fallback-on-client-registration-failure, defaulting to true, to allow the automatic fallback to default client_id / client_secret if OpenID Connect Dynamic Client Registration fails with any error * - fix infinite OAuth2 token refresh loop via SDK update * removed ipa name to get a timestamp ipa output name * changed ipa name by value * output git branch, set git branch in IPA file name * remove newline from gitbranch name * set git branch in branding.plist * renamed git hash key * - new version number for upload - changed changelog folder name * Calens changelog updated * [fix/connection-stuck] Fix eternal "Connecting…" (#1080) * - update SDK to add support for authentication-oauth2.oidc-fallback-on-client-registration-failure, defaulting to true, to allow the automatic fallback to default client_id / client_secret if OpenID Connect Dynamic Client Registration fails with any error * - fix infinite OAuth2 token refresh loop via SDK update * - BookmarkViewController: store user.displayName in .userDisplayName rather than .displayName - OCBookmark+AppExtensions: - make .displayName readonly and mark it as legacy - return .userDisplayName for .displayName if the former has a value - StaticLoginSingleAccountServerListViewController: - remove OCConnection code to fetch a user's display name and use bookmark.displayName instead - listen for bookmark updates and trigger a header update then accordingly - update SDK to gain bookmark.userDisplayName support * - update SDK to gain bookmark.userDisplayName support Co-authored-by: Matthias Hühne <[email protected]> * new build number * use latest develop sdk commit * [fix/core-overlap] Fix OCCore overlap for first connection (#1081) * - update SDK to add support for authentication-oauth2.oidc-fallback-on-client-registration-failure, defaulting to true, to allow the automatic fallback to default client_id / client_secret if OpenID Connect Dynamic Client Registration fails with any error * - fix infinite OAuth2 token refresh loop via SDK update * - BookmarkViewController: store user.displayName in .userDisplayName rather than .displayName - OCBookmark+AppExtensions: - make .displayName readonly and mark it as legacy - return .userDisplayName for .displayName if the former has a value - StaticLoginSingleAccountServerListViewController: - remove OCConnection code to fetch a user's display name and use bookmark.displayName instead - listen for bookmark updates and trigger a header update then accordingly - update SDK to gain bookmark.userDisplayName support * - update SDK to gain bookmark.userDisplayName support * Prevent unwanted connection following setup of the first account, preventing an immediate OCCore request and unclean return with possible instance overlap by starting the connection on the - at that time - already removed parent view controller. * new build number Co-authored-by: Matthias Hühne <[email protected]> * - updated build number - updated release notes - changed changelog folder name * Calens changelog updated * - update SDK (#1064) - make SortMethod.comparator use OCSQLiteCollationLocalized.sortComparator * Configuration documentation updated * added improved sorting to changelog * new changelog entry * Calens changelog updated Co-authored-by: hosy <[email protected]> Co-authored-by: Felix Schwarz <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR changes sorting to use localized sorting across query results and database queries, via the SDK's new
OCLOCALIZEDcollation and sort comparator.What was previously
1,100,11now becomes
1,11,100.Related Issue
#975
Types of changes