Skip to content

[INTPROD-9639] Add reaction payload type support#22

Merged
david-lyft merged 3 commits into
masterfrom
INTPROD-9639
May 29, 2025
Merged

[INTPROD-9639] Add reaction payload type support#22
david-lyft merged 3 commits into
masterfrom
INTPROD-9639

Conversation

@david-lyft
Copy link
Copy Markdown
Contributor

@david-lyft david-lyft commented May 28, 2025

  • Modified message_router to support reaction payload types
  • Updated python version as 3.8 is no longer supported in the setup environment

@david-lyft david-lyft requested review from colinsl and Copilot May 28, 2025 23:28
Copy link
Copy Markdown

Copilot AI left a comment

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 adds support for reaction payload types in the omnibot message router. The key changes include:

  • Adding a new reaction route test in router_test.py.
  • Updating the message handling condition in router.py to route both 'message' and 'reaction' payload types.
  • Expanding the routes dictionary and documentation to include the new 'reaction' match type.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
tests/unit/omnibot_receiver/router_test.py Introduces a test case to validate reaction payload type routing.
omnibot_receiver/router.py Updates the event handling condition and routes configuration to support reactions.

Comment thread omnibot_receiver/router.py Outdated
@david-lyft david-lyft merged commit 2ba97ae into master May 29, 2025
3 checks passed
@david-lyft david-lyft deleted the INTPROD-9639 branch May 29, 2025 01:00
david-lyft added a commit that referenced this pull request May 30, 2025
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.

3 participants