[PR #223] [MERGED] Show per-platform chat connection status in expanded overlay #225

New issue

Closed

opened 2026-03-02 04:00:49 +03:00 by kerem · 0 comments

kerem commented

2026-03-02 04:00:49 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/eerimoq/moblin/pull/223
Author: @Copilot
Created: 2/8/2026
Status: ✅ Merged
Merged: 2/8/2026
Merged by: @eerimoq

Base: main ← Head: copilot/update-chat-status-connection

📝 Commits (4)

d8352d3 Initial plan
5597075 Show per-platform chat connection status in expanded UI view
b6fdc1c Refactoring.
904a1a0 Refactoring.

📊 Changes

3 files changed (+84 additions, -9 deletions)

View changed files

📝 Moblin/Various/Model/Model.swift (+6 -0)
📝 Moblin/Various/Model/ModelChat.swift (+29 -3)
📝 Moblin/View/Stream/Overlay/StreamOverlayLeftView.swift (+49 -6)

📄 Description

Chat status showed a single Connected/Disconnected for all platforms combined. Viewers status already showed per-platform breakdown. This aligns chat status to match.

Changes

Model (Model.swift): Added ChatPlatformStatus struct and chatPlatformStatuses array to StatusTopLeft
Status logic (ModelChat.swift): Added updateChatPlatformStatuses() building per-platform connected state for each configured chat (Twitch, Kick, YouTube, SOOP, OSP, DLive). Skipped in remote control mode.
View (StreamOverlayLeftView.swift): Added ChatStatusView mirroring ViewersView — icon + per-platform logos with Connected/Disconnected text. Compact view unchanged (white/red icon only).

Behavior matrix

Mode	Expanded	Compact
Normal	Per-platform logos + status	White/red icon
Remote control	Text: "Connected/Disconnected (remote control)"	White/red icon
Not configured	Text: "Not configured"	White icon

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

www.swift.org

Triggering command: /usr/share/swift/usr/bin/swift-build /usr/share/swift/usr/bin/swift-build -c release (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Configure Actions setup steps to set up my environment, which run before the firewall is enabled

Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/eerimoq/moblin/pull/223 **Author:** [@Copilot](https://github.com/apps/copilot-swe-agent) **Created:** 2/8/2026 **Status:** ✅ Merged **Merged:** 2/8/2026 **Merged by:** [@eerimoq](https://github.com/eerimoq) **Base:** `main` ← **Head:** `copilot/update-chat-status-connection` --- ### 📝 Commits (4) - [`d8352d3`](https://github.com/eerimoq/moblin/commit/d8352d379194a42aa1352185fcaf87ce0bb4b4e7) Initial plan - [`5597075`](https://github.com/eerimoq/moblin/commit/55970757a3c4927e776f9eb72cd54984bfd22db6) Show per-platform chat connection status in expanded UI view - [`b6fdc1c`](https://github.com/eerimoq/moblin/commit/b6fdc1cfc4610484ab640466a0b94d7519c8c0c6) Refactoring. - [`904a1a0`](https://github.com/eerimoq/moblin/commit/904a1a0740cedc34376227cdacef3a52f8ae315c) Refactoring. ### 📊 Changes **3 files changed** (+84 additions, -9 deletions) <details> <summary>View changed files</summary> 📝 `Moblin/Various/Model/Model.swift` (+6 -0) 📝 `Moblin/Various/Model/ModelChat.swift` (+29 -3) 📝 `Moblin/View/Stream/Overlay/StreamOverlayLeftView.swift` (+49 -6) </details> ### 📄 Description Chat status showed a single Connected/Disconnected for all platforms combined. Viewers status already showed per-platform breakdown. This aligns chat status to match. ### Changes - **Model** (`Model.swift`): Added `ChatPlatformStatus` struct and `chatPlatformStatuses` array to `StatusTopLeft` - **Status logic** (`ModelChat.swift`): Added `updateChatPlatformStatuses()` building per-platform connected state for each configured chat (Twitch, Kick, YouTube, SOOP, OSP, DLive). Skipped in remote control mode. - **View** (`StreamOverlayLeftView.swift`): Added `ChatStatusView` mirroring `ViewersView` — icon + per-platform logos with Connected/Disconnected text. Compact view unchanged (white/red icon only). ### Behavior matrix | Mode | Expanded | Compact | |------|----------|---------| | Normal | Per-platform logos + status | White/red icon | | Remote control | Text: "Connected/Disconnected (remote control)" | White/red icon | | Not configured | Text: "Not configured" | White icon | > [!WARNING] > > <details> > <summary>Firewall rules blocked me from connecting to one or more addresses (expand for details)</summary> > > #### I tried to connect to the following addresses, but was blocked by firewall rules: > > - `www.swift.org` > - Triggering command: `/usr/share/swift/usr/bin/swift-build /usr/share/swift/usr/bin/swift-build -c release` (dns block) > > If you need me to access, download, or install something from one of these locations, you can either: > > - Configure [Actions setup steps](https://gh.io/copilot/actions-setup-steps) to set up my environment, which run before the firewall is enabled > - Add the appropriate URLs or hosts to the custom allowlist in this repository's [Copilot coding agent settings](https://github.com/eerimoq/moblin/settings/copilot/coding_agent) (admins only) > > </details>  --- ✨ Let Copilot coding agent [set things up for you](https://github.com/eerimoq/moblin/issues/new?title=✨+Set+up+Copilot+instructions&body=Configure%20instructions%20for%20this%20repository%20as%20documented%20in%20%5BBest%20practices%20for%20Copilot%20coding%20agent%20in%20your%20repository%5D%28https://gh.io/copilot-coding-agent-tips%29%2E%0A%0A%3COnboard%20this%20repo%3E&assignees=copilot) — coding agent works faster and does higher quality work when set up for your repo. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>