Skip to content

Add omit_channels option to userBoot - #4

Merged
KishanBagaria merged 2 commits into
beeperfrom
kb/dm-only
Aug 7, 2026
Merged

Add omit_channels option to userBoot#4
KishanBagaria merged 2 commits into
beeperfrom
kb/dm-only

Conversation

@KishanBagaria

Copy link
Copy Markdown
Member

Allow callers to set omit_channels on client.userBoot. Needed for local-signal without loading unnecessary Slack channel data.

Copilot AI lite review requested due to automatic review settings August 7, 2026 17:31

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR extends the client.userBoot call path to allow callers to omit channel data via a new omit_channels option, supporting use cases (like local-signal) that don’t need full channel payloads.

Changes:

  • Added ClientUserBootOptions to carry MinUpdated and the new OmitChannels flag.
  • Updated ClientUserBootContext to delegate to a new ClientUserBootWithOptionsContext helper.
  • Wired the omit_channels request parameter to opts.OmitChannels and updated min_channel_updated to use opts.MinUpdated.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread users.go Outdated

@tulir tulir left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The test failures can be ignored

@tulir tulir changed the title Add omit_channels option Add omit_channels option to userBoot Aug 7, 2026
@KishanBagaria
KishanBagaria merged commit 40f7674 into beeper Aug 7, 2026
0 of 4 checks passed
@KishanBagaria
KishanBagaria deleted the kb/dm-only branch August 7, 2026 18:04
@indent

indent Bot commented Aug 7, 2026

Copy link
Copy Markdown

Issues

Review closed.

CI Checks

4 CI checks failed on 2d599c1.

Failing lint go-1.25
  • lint go-1.25 finished with cancelled.
Failing lint go-1.26
  • lint go-1.26 finished with failure.
Failing test go-1.25
  • test go-1.25 finished with failure.
Failing test go-1.26
  • test go-1.26 finished with cancelled.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants