Skip to content
This repository was archived by the owner on Sep 3, 2025. It is now read-only.

fix(scheduler): use ast instead of json library to load string data #5275

Merged
whitdog47 merged 2 commits into from
Oct 1, 2024
Merged

fix(scheduler): use ast instead of json library to load string data #5275

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
13ad841
Using ast instead of json package to load string data
whitdog47 Oct 1, 2024
00177f9
removing unused library
whitdog47 Oct 1, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions src/dispatch/plugins/dispatch_slack/feedback/interactive.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
import logging
import json
import ast

from blockkit import (
Checkboxes,
Context,
Expand Down Expand Up @@ -404,7 +405,7 @@ def handle_oncall_shift_feedback_submission_event(
reminder_service.delete(db_session=db_session, reminder_id=reminder_id)
if len(metadata) > 5:
# if there are other details, store those
details = json.loads(metadata[5])
details = ast.literal_eval(metadata[5])
else:
details = None

Expand Down