Fix small IBKR Gateway dialog dismissal#71
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 530922d6e0
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| DISMISS_SMALL_GATEWAY_DIALOGS = os.environ.get( | ||
| "IBKR_DISMISS_SMALL_GATEWAY_DIALOGS", | ||
| "yes", |
There was a problem hiding this comment.
Forward the new dismissal toggle into the container
In the normal deployment path the bot runs inside the compose service, but docker-compose.yml uses an explicit environment: list with no env_file, and the workflow env writer only emits the existing 2FA variables. As a result, setting IBKR_DISMISS_SMALL_GATEWAY_DIALOGS=no in the host .env or repo variables never reaches 2fa_bot.py, so operators cannot disable this new broad IBKR Gateway dialog dismissal without editing the compose file itself.
Useful? React with 👍 / 👎.
Summary
Testing