Channel Actions - Only show Leave Group options if user has "leave-channel" permission

[bpollman]

🔗 Issue Link

Jira or Github issue link, if applicable.

🎯 Goal

Hide the " Leave Group" button/actions if the user doesnt have the permissions.

🛠 Implementation

NOTE: Depends on update to mocks main swift rep: GetStream/stream-chat-swift#3478

The code checks for"delete-channel" but not "leave-channel"

This new code

    public var shouldShowLeaveConversationButton: Bool {
        if channel.isDirectMessageChannel {
            return channel.ownCapabilities.contains(.deleteChannel)
        } else {
            return channel.ownCapabilities.contains(.leaveChannel)
        }
    }

properly reflect the existing logic that sets the leave/delete title here.

    public var leaveButtonTitle: String {
        if channel.isDirectMessageChannel {
            return L10n.Alert.Actions.deleteChannelTitle
        } else {
            return L10n.Alert.Actions.leaveGroupTitle
        }
    }

🧪 Testing

Describe the steps how this change can be tested (or why it can't be tested).

I've tested this with DM's, and channel types with and without the permissions

🎨 Changes

Add relevant screenshots or videos showcasing the changes.

☑️ Checklist

• I have signed the Stream CLA (required)
• Changelog is updated with client-facing changes
• New code is covered by unit tests
• Affected documentation updated (docusaurus, tutorial, CMS (task created)

[nuno-vieira]

Thank you for the PR @bpollman,

If you have the time, would be nice if you could update the shouldShowLeaveConversationButton tests with the new logic 👍

Otherwise, we will pick it up as soon as we can.

Best,
Nuno

[bpollman]

Thank you for the PR @bpollman,

If you have the time, would be nice if you could update the shouldShowLeaveConversationButton tests with the new logic 👍

Otherwise, we will pick it up as soon as we can.

Best, Nuno

Hey no problem,

I had to create another PR to update ChatChannel.mockDMChannel() to support channel capabilities. Package change is in a separate commit that can changed once the other PR is merged.