-
Notifications
You must be signed in to change notification settings - Fork 3.1k
[FEATURE REQUEST] Add a member to a space: Search user and show the list of matches #4754
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
base: master
Are you sure you want to change the base?
Conversation
7299fa1 to
27030d9
Compare
c1041e3 to
7db90f9
Compare
3339374 to
809857b
Compare
owncloudApp/src/main/java/com/owncloud/android/presentation/spaces/members/AddMemberFragment.kt
Outdated
Show resolved
Hide resolved
owncloudApp/src/main/java/com/owncloud/android/presentation/spaces/members/AddMemberFragment.kt
Show resolved
Hide resolved
...udApp/src/main/java/com/owncloud/android/presentation/spaces/members/SearchMembersAdapter.kt
Outdated
Show resolved
Hide resolved
...udApp/src/main/java/com/owncloud/android/presentation/spaces/members/SpaceMembersActivity.kt
Outdated
Show resolved
Hide resolved
...oudApp/src/main/java/com/owncloud/android/presentation/spaces/members/SpaceMembersAdapter.kt
Show resolved
Hide resolved
...va/com/owncloud/android/data/members/datasources/implementation/OCRemoteMembersDataSource.kt
Outdated
Show resolved
Hide resolved
owncloudData/src/main/java/com/owncloud/android/data/members/repository/OCMembersRepository.kt
Outdated
Show resolved
Hide resolved
owncloudData/src/test/java/com/owncloud/android/data/members/OCRemoteMembersDataSourceTest.kt
Outdated
Show resolved
Hide resolved
...om/owncloud/android/data/members/datasources/implementation/OCRemoteMembersDataSourceTest.kt
Show resolved
Hide resolved
owncloudData/src/test/java/com/owncloud/android/data/members/OCRemoteMembersDataSourceTest.kt
Outdated
Show resolved
Hide resolved
0994368 to
0f38db8
Compare
4c1b022 to
c3afb33
Compare
jesmrec
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CR approved, let's QA this
(1)Typing characters does not refresh screen with results. App should be unfocused and focused to list: Screenrecorder-2026-01-21-13-31-54-142.mp4NOTE: not in every server... crazy Xiaomi Redmi Note 13, Android 15
|
(2) [FIXED]In search view, add some special characters like Current: Snackbar I realised the query parameter is Expected: No error, just no results in case there are no matches. Checking iOS, the query parameter is enclosed in quotation marks (if it helps):
Xiaomi Redmi Note 13, Android 15
|
(3)In case the typed chars don't match with any user, it'd be a good idea to show a message like Xiaomi Redmi Note 13, Android 15
|
|
I think that (1) and (2) are fixed now @jesmrec 🙌🏻 |
(4) (improvement)Given a high number of results, the lag for showing the results is perceptible. Could it be posible to add a spinner, so the view does not seem to be frozen. Ideally (it might not be feasible), the spinner would replace the ✖️ while the operation is executed. Xiaomi Redmi Note 13, Android 15
|
|
(2) fixed |
Related Issues
App: #4613
ReleaseNotesViewModel.ktcreating a newReleaseNote()with String resources (if required)QA
Test plan:
Reports:
Unknown errorwhen special characters [FEATURE REQUEST] Add a member to a space: Search user and show the list of matches #4754 (comment) [FIXED]