appsmith icon indicating copy to clipboard operation
appsmith copied to clipboard

[Bug]: Table Row Frontend Desync with other Components

Open mw-risRX opened this issue 1 year ago • 14 comments

Is there an existing issue for this?

  • [X] I have searched the existing issues

Description

When Users click a row, the information that populates forms/other components on the same page displays information from a different row. The users sometimes do not recognize this until they submit the 'forms' and realize the incorrect row has been updated with the desync'd information. Users essentially end up updating the wrong row , their "Selected Row" is not always accurate. We tried fixing code-wise/putting safety nets and found that when calling [tableComponent].selectedRow it would be reflecting the information that is shown on the forms, and no record of the 'selectedRow' thats visually highlighted when onClick. So code wouldn't see what user was seeing.

Steps To Reproduce

  1. Navigate to our home 'page'
  2. spam click between a few rows
  3. the forms/components below are supposed to sync with the rows

// we had a video demonstrating but let us know if you can reach out we can provide a secure video

Public Sample App

No response

Environment

Production

Severity

Critical (Broken Production apps)

Issue video log

No response

Version

Appsmith Community v1.28

mw-risRX avatar Aug 12 '24 17:08 mw-risRX

Hello,

We've been talking with appsmith support ([email protected]) about this issue and were told to create a bug report as they were unable to reproduce the error on their side.

For additional detail, this issue has persisted across multiple versions for us so we're not sure if just updating our version will help

Looking forward to hearing back, thank you!

dc-risrx avatar Aug 12 '24 18:08 dc-risrx

@dc-risrx we would like to have a debugging session for this. Could you schedule a call according to your availability using https://calendly.com/rishabh-rathod

rishabhrathod01 avatar Aug 13 '24 16:08 rishabhrathod01

Hello @rishabhrathod01 sounds good just scheduled for this Friday at 8AM PST, looking forward to meeting thank you very much!

dc-risrx avatar Aug 13 '24 16:08 dc-risrx

Hey @dc-risrx, Pardon for a miss from my side. Friday was supposed to be marked as OOO on my calendar but wasn’t updated, i have done that now. I request you to reschedule the meeting using https://calendly.com/reschedulings/0ef5c3de-185a-424f-8d1e-2862321c1526, Thanks.

rishabhrathod01 avatar Aug 13 '24 19:08 rishabhrathod01

no worries @rishabhrathod01 just rescheduled!

dc-risrx avatar Aug 13 '24 19:08 dc-risrx

This critical issue has not seen activity for a while. It will be closed in 7 days unless further activity is detected or the Critical tag is removed.

github-actions[bot] avatar Aug 21 '24 16:08 github-actions[bot]

hello @rishabhrathod01 apologies, not sure if you saw my email this morning and I just saw the calendly options to reschedule or cancel but I'm unable to access those links for some reason

we were having some emergencies with one of our systems this morning so so sorry again that we had to miss, would you be able to resend a calendly invitation please so that we can reschedule

dc-risrx avatar Aug 21 '24 17:08 dc-risrx

@dc-risrx No worries, you could schedule a new call using calendly.com/rishabh-rathod

rishabhrathod01 avatar Aug 21 '24 20:08 rishabhrathod01

thank you so much, just sent it

apologies again!

dc-risrx avatar Aug 21 '24 20:08 dc-risrx

@dc-risrx we expected to receive an email with the details but haven't yet received one. Also, do add [email protected] to the email with [email protected]. Thanks

rishabhrathod01 avatar Aug 26 '24 10:08 rishabhrathod01

hello @rishabhrathod01, apologies took us longer to get this out than we wanted but just sent it over! please let us know if there's any issues, thank you!

dc-risrx avatar Aug 26 '24 15:08 dc-risrx

@dc-risrx The application JSON and the data schema didn't help us reproduce the issue. We will need to analyze the application further and would request to schedule a call for the same. Could you please use this link to schedule a call again calendly.com/rishabh-rathod

rishabhrathod01 avatar Aug 29 '24 10:08 rishabhrathod01

This critical issue has not seen activity for a while. It will be closed in 7 days unless further activity is detected or the Critical tag is removed.

github-actions[bot] avatar Sep 07 '24 16:09 github-actions[bot]

this should still be treated as a critical issue - we are working on another critical issue( #35523 ) before coming to this

rohan-arthur avatar Sep 09 '24 09:09 rohan-arthur

@dc-risrx We created the query to trigger onRowSelection but unfortunately, this issue is not yet reproducible on our side. We noticed an await used in the binding of onRowSelection. That await could be skipped. We will keep updating here with more information.

rishabhrathod01 avatar Sep 13 '24 01:09 rishabhrathod01

@dc-risrx After further analysis, we confirmed that it is an old bug that got fixed with this PR https://github.com/appsmithorg/appsmith/pull/34106 and is available in version 1.29. We would suggest you to please upgrade to get the fix. I will be closing the issue for now. Please reopen if it is still reproducible. Thanks for your patience.

rishabhrathod01 avatar Sep 13 '24 11:09 rishabhrathod01