Results 53 comments of Mehul

[no show status conflict update.webm](https://github.com/user-attachments/assets/02d0036a-e2e2-45a6-bf44-339ac75a3295) As we have moved to tanstack table, we have to batch the mutations of attendees in the noShowAttendeesDialog. Otherwise a single toggle mutation will reset(update...

@Praashh Final touch as in review or a nit?

> @MehulZR can you fix the conflicts , so it can be taken forward , btw LGTM Done

@TusharBhatt1 I am not sure why you went ahead with #18648 as it contains some issues. I have left some related comments there.

> Also can you have a look at this : https://www.loom.com/share/e852a67beace463693eedbeb8d043dd2 Wasn't able to reproduce on Chrome and Firefox. Can you provide steps to reproduce this? https://github.com/user-attachments/assets/0cf3e72a-8804-4ec0-943d-1d4a4cfeabb5 Notice when I...

> even the drop down immediately closes after it gets open Yeah, that also stood out to me too. Can you still reproduce it?

> @MehulZR I can still see closing of dropdown as soon as it opens and unnecessary network calls , mentioned above - https://www.loom.com/share/e852a67beace463693eedbeb8d043dd2 I will do some testing by freshly...

> When no updation and user clicks Done , don't show any alert. This is subjective but generally end user expects a response when they interact with something. The popup...

My bad. My explanation wasn't very clear. What I meant was I haven't changed the message returned by the backend, which is used to show the pop up notification. >...

https://github.com/user-attachments/assets/2c0989be-86f8-4179-aebe-c3106a9a8d30 This is on main branch. Grouped Attendees have the same behavior. 1. It mutates even if there isn't any change 2. Doesn't triggers the notification individually Do we want...