title change event reaction kind of works, taking a lunch break#214
title change event reaction kind of works, taking a lunch break#214rajeshrah22 wants to merge 6 commits intomainfrom
Conversation
|
Build check completed! View branch preview here: https://rahul-task-fix.d3c5lsis3camij.amplifyapp.com/. Please allow 10-15 minutes for the preview to be deployed. Deployment errors will not be shown here. |
…itle and detail fields
|
Build check completed! View branch preview here: https://rahul-task-fix.d3c5lsis3camij.amplifyapp.com/. Please allow 10-15 minutes for the preview to be deployed. Deployment errors will not be shown here. |
|
Build check completed! View branch preview here: https://rahul-task-fix.d3c5lsis3camij.amplifyapp.com/. Please allow 10-15 minutes for the preview to be deployed. Deployment errors will not be shown here. |
|
Build check completed! View branch preview here: https://rahul-task-fix.d3c5lsis3camij.amplifyapp.com/. Please allow 10-15 minutes for the preview to be deployed. Deployment errors will not be shown here. |
…e I wish I had someone knowledgable to review our code :/
|
Build check completed! View branch preview here: https://rahul-task-fix.d3c5lsis3camij.amplifyapp.com/. Please allow 10-15 minutes for the preview to be deployed. Deployment errors will not be shown here. |
| newMap.set(task.id, task); | ||
| } | ||
|
|
||
| setIsLoading(false); |
There was a problem hiding this comment.
Do we want to be raising this flag here?
2 participants
Overall Review of Changes:
A general description of the changes made that are being requested for merge
Reason for Change
But why?
Tested:
Yes/No/NA