Skip to content

Commit 0a60cce

Browse files
committed
trigger processing correctly
1 parent 0214e76 commit 0a60cce

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

packages/slackBotFunction/app/slack/slack_events.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
)
2626
from app.services.query_reformulator import reformulate_query
2727
from app.services.slack import get_friendly_channel_name
28-
from app.utils.handler_utils import is_duplicate_event
28+
from app.utils.handler_utils import extract_pull_request_id, is_duplicate_event
2929

3030
logger = get_logger()
3131

@@ -276,6 +276,9 @@ def process_pull_request_slack_event(slack_event_data: Dict[str, Any]) -> None:
276276
event_id = slack_event_data["event_id"]
277277
if is_duplicate_event(event_id=event_id):
278278
return
279+
message_text = slack_event_data["event"]["text"]
280+
_, extracted_message = extract_pull_request_id(message_text)
281+
slack_event_data["event"]["text"] = extracted_message
279282
process_async_slack_event(slack_event_data=slack_event_data)
280283

281284

packages/slackBotFunction/app/slack/slack_handlers.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
from app.services.dynamo import get_state_information
2424
from app.utils.handler_utils import (
2525
conversation_key_and_root,
26+
extract_pull_request_id,
2627
gate_common,
2728
is_latest_message,
2829
strip_mentions,
@@ -271,7 +272,7 @@ def _common_message_handler(
271272

272273
if message_text.lower().startswith(constants.PULL_REQUEST_PREFIX):
273274
try:
274-
pull_request_id, extracted_message = _extract_pull_request_id(message_text)
275+
pull_request_id, _ = extract_pull_request_id(message_text)
275276
trigger_pull_request_processing(pull_request_id=pull_request_id, event=event, event_id=event_id)
276277
except Exception as e:
277278
logger.error(f"Can not find pull request details: {e}", extra={"error": traceback.format_exc()})

0 commit comments

Comments
 (0)