fix(mobile): resolve frontend bug bundle (#1209)#273
Conversation
Test Results20 tests 20 ✅ 1s ⏱️ Results for commit e273edd. ♻️ This comment has been updated with latest results. |
|
|
Moved the note length validation out of |
|
|
||
| return await apiClient.post(url, { text }); | ||
| const trimmedText = text.trim(); | ||
|
|
There was a problem hiding this comment.
I think the text has already been trimmed before passing into this service, so you don't need to trim it here again.
Also, you don't need to add the try-catch block here because the return await will handle the try-catch error automatically.
jen12138
left a comment
jen12138
left a comment
There was a problem hiding this comment.
Updated.
I removed the extra trim and try/catch in notes-service.ts as suggested. The note text is already handled before calling the service, and mutation errors are already surfaced through the existing React Query + toast flow.
…d/Blotz-Task-App into fix/1209-frontend-bugs
…d/Blotz-Task-App into fix/1209-frontend-bugs
4 participants
Done Fix Frontend Bugs (Language Update, Task Type Tab, Note Modal Behavior, Note Length Validation)