tiennm99/tiennm99bot
1
0 Fork 0
mirror of https://github.com/tiennm99/tiennm99bot.git synced 2026-10-15 22:15:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(deploy): include stats in CI ModulesCSV override

Browse Source 
SAM CLI's --parameter-overrides replaces samconfig.toml's overrides
rather than merging them, so every CI deploy reset the stack's
ModulesCSV to the prior value (which predated the stats module).
The live Lambda's MODULES env was missing stats, so /stats was
dispatched but never handled — the module factory was never called.
This commit is contained in:
tiennm99 committed 2026-05-22 17:27:26 +07:00
1 parent 819d4a985e
commit cf7b8366b3
1 file changed
+1 -1
Download Patch File Download Diff File Expand all files Collapse all files
.github/workflows/deploy.yml
+1 -1
View File
@@ -59,7 +59,7 @@ jobs:
# CI needs in the deployed stack must be listed here explicitly.
# Telegram user IDs are public (visible to anyone the bot DMs), so
# they live in this committed workflow rather than a secret.
OVERRIDES="CronSharedSecret=$CRON_SECRET BotOwnerID=1064111334 AdminUserIDs=1064111334"
OVERRIDES="CronSharedSecret=$CRON_SECRET BotOwnerID=1064111334 AdminUserIDs=1064111334 ModulesCSV=util,misc,wordle,loldle,lolschedule,twentyq,trading,stats"
if [ -n "$ALERT_EMAIL" ]; then
OVERRIDES="$OVERRIDES AlertEmail=$ALERT_EMAIL"
fi