cal/major-domo-v2
1
0
Fork 0
Code Issues 6 Pull Requests 2 Actions Packages Projects Releases Wiki Activity

fix: add missing logger to SubmitConfirmationModal

Browse Source 
The error handler at line 384 referenced self.logger but __init__
never set it, causing AttributeError that silently swallowed the
real error from users.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Cal Corum 2026-04-02 08:30:24 -05:00
parent c8cb80c5f3
commit 0ea1c1d633
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
views/transaction_embed.py
View File

@ -6,6 +6,8 @@ Handles the Discord embed and button interfaces for the transaction builder.
import discord
from utils.logging import get_contextual_logger
from services.transaction_builder import (
TransactionBuilder,
clear_transaction_builder,
@ -235,6 +237,7 @@ class SubmitConfirmationModal(discord.ui.Modal):
super().__init__(title="Confirm Transaction Submission")
self.builder = builder
self.submission_handler = submission_handler
self.logger = get_contextual_logger(f"{__name__}.SubmitConfirmationModal")
self.confirmation = discord.ui.TextInput(
label="Type 'CONFIRM' to submit",