Skip to content

fix: wrap addTask, updateTask and deleteTask in try/catch to prevent … - #105

Merged
anoopcodehack merged 1 commit into
anoopcodehack:mainfrom
omkarwarik02:fix/addtask-updatetask-error-handling
Jul 30, 2026
Merged

fix: wrap addTask, updateTask and deleteTask in try/catch to prevent …#105
anoopcodehack merged 1 commit into
anoopcodehack:mainfrom
omkarwarik02:fix/addtask-updatetask-error-handling

Conversation

@omkarwarik02

Copy link
Copy Markdown

Closes #97

Changes

  • Wrapped addTask, updateTask, and deleteTask in try/catch in BoardContext.jsx
  • Errors are logged via console.error and re-thrown so calling components can catch them and show a toast/error message
  • Included deleteTask as a bonus fix (same pattern) as suggested by the maintainer

Testing

  • Verified the app loads and doesn't crash when the backend is unreachable
  • Reviewed all callers of addTask/updateTask/deleteTask to confirm no breaking changes

@anoopcodehack
anoopcodehack merged commit cdba107 into anoopcodehack:main Jul 30, 2026
@anoopcodehack

Copy link
Copy Markdown
Owner

Hey, your PR has been merged! You're officially a DevBoard contributor now!

If you enjoyed contributing and found the project useful, consider leaving a ⭐ on the repo — it really helps more developers discover the project and keeps the community growing!

Also feel free to pick up another issue anytime — always happy to have you here! 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AddTask and UpdateTask crash silently on API error➕

2 participants