Skip to content

Fix webhook remove#30

Merged
tulir merged 1 commit intomaubot:masterfrom
OliveiraHermogenes:fix-webhook-remove
Apr 5, 2026
Merged

Fix webhook remove#30
tulir merged 1 commit intomaubot:masterfrom
OliveiraHermogenes:fix-webhook-remove

Conversation

@OliveiraHermogenes
Copy link
Copy Markdown
Contributor

The arguments to get_by_repo were being passed in the wrong order.

async def get_by_repo(self, room_id: RoomID, repo: str) -> WebhookInfo | None:
return self._add_to_cache(await self._db.get_webhook_by_repo(room_id, repo))

@OliveiraHermogenes
Copy link
Copy Markdown
Contributor Author

A screenshot of the behavior before the fix:

image

Copy link
Copy Markdown
Member

@tulir tulir left a comment

Choose a reason for hiding this comment

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

Not sure how that wasn't noticed before 🤔

@tulir tulir merged commit 4e1af25 into maubot:master Apr 5, 2026
1 check failed
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