-
Notifications
You must be signed in to change notification settings - Fork 19
title change event reaction kind of works, taking a lunch break #214
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to be raising this flag here?
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