rowboat
rowboat copied to clipboard
Better error handling
@UjjwalTiwari00 is attempting to deploy a commit to the RowBoat Labs Team on Vercel.
A member of the Team first needs to authorize it.
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.
Hi @UjjwalTiwari00
Here are some observations from my review:
- This project uses already uses UI components from HeroUI, shadcn, etc. HeroUI also has a Toast component available. Can we use that instead of adding a new dependency?
- The idea behind https://github.com/rowboatlabs/rowboat/issues/142 is to also propagate certain errors, such as running out of OpenAI credits. Can we add the necessary changes for that as well? Currently this seems mostly like a UI change.
- Removing
setFetchResponseError('Error: ' + JSON.parse(event.data).error);from the chat component also removes the ability for the user to retry the action. We need to think of how to keep that ability while also using toast vs not using toast in certain UX flows. - This PR needs to be merged into
dev, notmain
Currently, the changes are only related to the look-and-feel. It would be sensible to also include changes related to propagation of meaningful errors from the backend and upstream services (like agents service / copilot, etc)