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.

Sign up for GitHub

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

Jump to bottom

Can't unban a user #1170

Closed
jeangb opened this issue Apr 20, 2017 · 2 comments
Closed

Can't unban a user #1170

jeangb opened this issue Apr 20, 2017 · 2 comments
Labels
auto-test-required P2 T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Milestone
sprint6

Comments

@jeangb
Copy link

jeangb commented Apr 20, 2017

To reproduce:

  1. Be admin in a room
  2. Ban a user
  3. Try to unban him

"Cannot leave user who was banned" alertdialog appears.
Related to element-hq/riot-android#938
The issue is corrected in riot-web and riot-android.
@jeangb jeangb added auto-test-required T-Defect Something isn't working: bugs, crashes, hangs and other reported problems P2 labels Apr 20, 2017
@ylecollen
Copy link

here is the new API
https://github.com/matrix-org/matrix-android-sdk/blob/master/matrix-sdk/src/main/java/org/matrix/androidsdk/rest/api/RoomsApi.java#L204

@giomfo giomfo added this to the sprint6 milestone May 2, 2017
giomfo added a commit to matrix-org/matrix-ios-sdk that referenced this issue Sep 8, 2017
@giomfo
Merge pull request #367 from ThibaultFarnier/feature/fix_unban_and_kick
e4f491b 
Fix API for unbanning and kicking
(Fix element-hq/element-ios#1170)
@giomfo
Copy link
Member

giomfo commented Sep 8, 2017

Closed, but auto-test is missing

@giomfo giomfo closed this as completed Sep 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
auto-test-required P2 T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Projects
None yet
Milestone
sprint6
Development

No branches or pull requests
3 participants
@giomfo @ylecollen @jeangb