-
Notifications
You must be signed in to change notification settings - Fork 3.1k
[FEATURE REQUEST] Remove a space member #4768
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
5ad44c8 to
71f5e34
Compare
...udApp/src/main/java/com/owncloud/android/presentation/spaces/members/SpaceMembersFragment.kt
Outdated
Show resolved
Hide resolved
...udApp/src/main/java/com/owncloud/android/presentation/spaces/members/SpaceMembersFragment.kt
Show resolved
Hide resolved
...rary/src/main/java/com/owncloud/android/lib/resources/members/RemoveRemoteMemberOperation.kt
Outdated
Show resolved
Hide resolved
...oudApp/src/main/java/com/owncloud/android/presentation/spaces/members/SpaceMembersAdapter.kt
Show resolved
Hide resolved
f05d479 to
5a288cf
Compare
|
moving to QA, since #4768 (comment) is out of CR scope. In case changes are required, will be handled as a QA report. |
(1)All steps to be done in the mobile app:
Xiaomi Redmi Note 13, Android 15 |
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, QA time
3d0e799 to
6863e3f
Compare
|
(1): This bug is reproducible after applying the commit that comes from this comment on the CR: #4768 (comment) I've checked it, and the best solution here is to revert all the changes and removed that commit. It's more efficient to use I've removed the commit and it should be fixed now 😄 |
Related Issues
App: #4725
ReleaseNotesViewModel.ktcreating a newReleaseNote()with String resources (if required)QA
Reports: