appsmith icon indicating copy to clipboard operation
appsmith copied to clipboard

[Bug]: currentRow is undefined Inside the Options Property of a Select Column of a Table Widget

Open ame-appsmith opened this issue 2 years ago • 1 comments

Is there an existing issue for this?

  • [X] I have searched the existing issues

Description

The user has an editable Select column inside a Table widget. If trying to use currentRow inside the Options property of the Select column, the error currentRow is undefined displays, hence it is not possible to conditionally set the Options of the Select column based on other values from the current row of the table. image

Steps To Reproduce

  1. Add a Table widget and populate it with data.
  2. Make one of the columns editable and set its type to Select.
  3. Try to use currentRow inside the Options field of the Select column settings.

Public Sample App

No response

Issue video log

No response

Version

Appsmith v1.8.7-SNAPSHOT

Front logo Front conversations

ame-appsmith avatar Nov 11 '22 15:11 ame-appsmith

Tested this Issue and working as expected

  • Tested currentRow functionality inside the options property of select column

chandannkumar avatar Dec 07 '22 06:12 chandannkumar

@chandannkumar it doesn't seem to be working for me. Which DP did you test it in?

Nikhil-Nandagopal avatar Dec 10 '22 07:12 Nikhil-Nandagopal

@Nikhil-Nandagopal I tested the fix on this DP: https://appsmith-git-feat-currentrow-access-select-options-get-appsmith.vercel.app/ The current vercel DP link seems broken now @sbalaji1192

chandannkumar avatar Dec 12 '22 04:12 chandannkumar

@chandannkumar triggered another vercel dp - https://appsmith-grse0j68l-get-appsmith.vercel.app/applications

sbalaji1192 avatar Dec 12 '22 04:12 sbalaji1192

@chandannkumar Could you perform a regression on this issue? I have made some changes to fix a failing cypress test case.

sbalaji1192 avatar Dec 12 '22 08:12 sbalaji1192

Re-Tested this issue and LGTM

chandannkumar avatar Dec 14 '22 04:12 chandannkumar