-
Notifications
You must be signed in to change notification settings - Fork 608
Mobile v2.36 Docs: Add self-management capability to channel members #8665
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
- Added documentation for managing own membership from Manage Channel Members screen - Reflects PR #9301 / MM-66375 bug fix in mobile v2.36+ - Mobile-specific workflow for users with member management permissions 🤖 Generated with [Claude Code](https://claude.ai/code) Co-authored-by: Combs7th <Combs7th@users.noreply.github.com>
|
@larkox - Do the above doc updates look accurate to you? |
|
Newest code from mattermost has been published to preview environment for Git SHA 197a9ff |
|
@Combs7th The documentation is technically correct, but I am on the fence on whether it is worth it to add this particular thing to the documentation. Feels more like clutter than anything else. My point is:
I am going to approve this, in case you want to move forward with this, but please keep in consideration my comments in case you want to close this PR. |
|
Newest code from mattermost has been published to preview environment for Git SHA c4d3484 |
|
@john-combs What if we were to combine the "Leave Channel" and "Manage Members" portion. Perhaps we can just add a note to the bottom of that section that states: If you have permission to manage channel members, you can also leave a channel from View Info → Members → Manage by selecting your name. This setting also allows you to manage your role if that permission is also granted |
Combs7th
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.
Updated doc as per the suggestion from @wiersgallak. I think I did that edit & commit correctly? 🤞🏻
|
Newest code from mattermost has been published to preview environment for Git SHA 87dd504 |
larkox
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.
LGTM
| .. image:: ../../images/mobile-exit-after-removing-user-from-a-channel.jpg | ||
| :alt: Tap on Done to exit and return to the channel. | ||
| :scale: 30 | ||
| :scale: 30 |
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.
There seems to be an unrelated change here.
wiersgallak
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.
Good to me, pending addressing larkox note
Updates documentation to reflect PR #9301 / MM-66375 bug fix in mobile v2.36+
Closes #8664
🤖 Generated with Claude Code