Merge pull request 'fix: add missing logger to SubmitConfirmationModal' (#123 ) from fix/submit-modal-missing-logger into main

Reviewed-on: #123
fix: add missing logger to SubmitConfirmationModal
2026-04-06 21:57:50 +00:00 · 2026-04-02 08:30:24 -05:00
1 changed files with 3 additions and 0 deletions
--- a/views/transaction_embed.py
+++ b/views/transaction_embed.py
@ -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",
Author	SHA1	Message	Date
cal	dd253dab09	Merge pull request 'fix: add missing logger to SubmitConfirmationModal' (#123 ) from fix/submit-modal-missing-logger into main Reviewed-on: #123	2026-04-06 21:57:50 +00:00
Cal Corum	0ea1c1d633	fix: add missing logger to SubmitConfirmationModal 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>	2026-04-02 08:30:24 -05:00