fix(webapp): scheduled task test page crash on empty payloads#3335
fix(webapp): scheduled task test page crash on empty payloads#3335itskrishna21 wants to merge 1 commit intotriggerdotdev:mainfrom
Conversation
Harden scheduled run payload parsing to handle empty/malformed packets without throwing, and add a regression test. Made-with: Cursor
|
|
Hi @itskrishna21, thanks for your interest in contributing! This project requires that pull request authors are vouched, and you are not in the list of vouched users. This PR will be closed automatically. See https://github.com/triggerdotdev/trigger.dev/blob/main/CONTRIBUTING.md for more details. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
WalkthroughThis PR fixes scheduled task test page crashes caused by empty payloads. The changes extract a Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Details
This addresses
refs #3316.Test plan
cd apps/webapp && pnpm run test ./test/scheduledTaskTestPageEmptyPayload.test.ts --runNotes
.server-changes/entry for release notes.Made with Cursor