Skip to content

connector: update GetChatInfoChange for current mautrix-go - #10

Merged
iFixRobots merged 1 commit into
mainfrom
iFixRobots/mautrix-go-chatinfochange-portal
Oct 27, 2026
Merged

iFixRobots merged 1 commit into
mainfrom
iFixRobots/mautrix-go-chatinfochange-portal

Conversation

@iFixRobots

Copy link
Copy Markdown
Contributor

megabridge's bridge update has been failing since #4 merged. The new blockListChange implements GetChatInfoChange(ctx), but the mautrix-go that megabridge builds against (b5763ae) now expects GetChatInfoChange(ctx, portal), so the build stops at:

pkg/connector/blocking.go:122:39: *blockListChange does not implement bridgev2.RemoteChatInfoChange (wrong type for method GetChatInfoChange)

That blocks every bridge bump in megabridge, not only Reddit. This PR bumps mautrix-go to megabridge's pin and adds the *bridgev2.Portal parameter, which blockListChange doesn't use. The existing test call passes nil for it.

View in Indent
Tag @indent to continue the conversation here.

@indent

indent Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Issues

No issues found.

CI Checks

All CI checks passed on fb69f75.

Review agents

Select any unchecked box below to run or rerun that agent.

Passed (1)
  • Indent Review Agent · No issues found; builds against the new pin.
Full results

Indent Review Agent

  • Summary: No issues found; builds against the new pin.
  • Last ran on commit: fb69f75b
  • Latest result
    {
      "summary": "No issues found; builds against the new pin.",
      "findings": []
    }

@iFixRobots
iFixRobots merged commit ce81b9c into main Oct 27, 2026
11 checks passed
@iFixRobots
iFixRobots deleted the iFixRobots/mautrix-go-chatinfochange-portal branch October 27, 2026 23:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant