Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@claude review and update if there any findings |
|
Claude finished @makeavish's task in 2m 54s —— View job Review & Fixes for PR #2688
Findings & Fixes Applied1. 🐛 Broken link (fixed): The Celery configuration link used 2. 📝 Improved section structure (fixed): The original used a numbered list where step 2 was just an explanation, not an action. Restructured to use clear prose with the command first, followed by an alternative approach. 3. ➕ Added config-file alternative (added): Users who prefer not to use CLI flags can set 4. 📅 Updated frontmatter date (fixed): The pre-commit hook flagged the outdated date ( Summary of Changes- ...which requires the `worker_send_task_events` setting to be enabled.
+ ...which requires the `worker_send_task_events` setting to be enabled.
- To enable worker online status:
- 1. Start your Celery worker with the `-E` flag...
- 2. This will allow the dashboard to properly track...
+ To fix this, start your Celery worker with the `-E` flag...
+ Alternatively, you can enable task events in your Celery configuration:
+ worker_send_task_events = True
+ Once enabled, the dashboard will properly track...
- ...configuration.html#worker_send_task_events)
+ ...configuration.html#worker-send-task-events)| |
- Fix broken link anchor (#worker_send_task_events -> #worker-send-task-events) - Add config-file alternative for enabling task events - Improve section structure and clarity - Update frontmatter date Co-authored-by: Vishal Sharma <makeavish@users.noreply.github.com> Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
738e0c5 to
512b0b1
Compare
Automated docs update from issue #2681.\n\nCloses #2681