Skip to content

Adding ContextTeamId, Updated, IsOpen, IsFrozen and the shared team ID lists to Conversation - #269

Merged
soxtoby merged 2 commits into
soxtoby:masterfrom
we4sz:conversation-team-ids-and-open-state
Nov 1, 2026
Merged

soxtoby merged 2 commits into
soxtoby:masterfrom
we4sz:conversation-team-ids-and-open-state

Conversation

@we4sz

@we4sz we4sz commented Oct 31, 2026

Copy link
Copy Markdown
Contributor

Eight fields that Slack documents on its conversation object have no property in SlackNet 0.19.0. I found the first three comparing a real conversations.open response (a DM, with return_im) with Conversation; the other five are the frozen and shared channel fields from the same page, which matter for Slack Connect channels. My app doesn't read any of them through a workaround today. This models them. It is two commits, each one builds and passes the tests.

  1. ContextTeamId (context_team_id): the workspace the conversation is within. Updated (updated): when the channel settings were updated, in milliseconds, so it is a long (Created is in seconds). IsOpen (is_open): whether a DM is open.
  2. IsFrozen (is_frozen), and SharedTeamIds, InternalTeamIds, ConnectedTeamIds and PendingConnectedTeamIds (shared_team_ids, internal_team_ids, connected_team_ids, pending_connected_team_ids). The lists are IList<string>, empty when Slack sends none, like Members.

The doc comments copy Slack's descriptions, as AGENTS.md asks.

Compatibility

Additive only. The new properties default to false, 0, null or an empty list.

Testing

New ConversationTests: a DM with the first three fields, a shared channel with the second group, and a plain channel with neither. ./do test passes on both commits (785 and 787 tests).

Seen in a real response: context_team_id, updated (13 digits, so milliseconds) and is_open are in the conversations.open response I captured, and the reference's conversations.open example shows is_open.

Not verified: is_frozen and the four team ID lists. I haven't seen them in a real response, so they follow the reference page and its one-line descriptions. I also haven't run my app against a local pack of this branch.

@we4sz
we4sz force-pushed the conversation-team-ids-and-open-state branch from 0600b5d to 13559d2 Compare October 31, 2026 08:43
@soxtoby
soxtoby merged commit c320512 into soxtoby:master Nov 1, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants