appsmith icon indicating copy to clipboard operation
appsmith copied to clipboard

[Bug]: Issue seen in an app if there is a query with a column name as BorderRadius

Open btsgh opened this issue 2 years ago • 5 comments

Is there an existing issue for this?

  • [X] I have searched the existing issues

Description

User cannot have query with column name as "BorderRadius"

Steps To Reproduce

This bug seems to be related to theming. Removing a column named "BorderRadius" solves the problem. Here are the videos showing the issue:

  1. Shows issue: LOOM DEMO
  2. Shows how the issue can be avoided by removing BorderRadius column LOOM DEMO

Public Sample App

https://release.app.appsmith.com/app/habitasaurus/profile-6268e911bd7afb15ac39ba0b/edit#50cdgqv7rr

Version

Release

btsgh avatar May 05 '22 13:05 btsgh

Need access to this app, it says not found for me. Please share it with [email protected]

dilippitchika avatar May 05 '22 14:05 dilippitchika

@btsgh removing critical tag and using release blocker.

Please use critical if something breaks in production, I am assuming this issue doesn't happen in production yet.

dilippitchika avatar May 05 '22 14:05 dilippitchika

@btsgh I am not able to reproduce this issue on my end :/

jsartisan avatar May 06 '22 07:05 jsartisan

@jsartisan - In the Habitasaurus app, Profile page, Apply Theme gives the uncaught promise rejection if the GetThemes query has this Select statement - SELECT * FROM "Themes" ORDER BY id LIMIT 10;

If the query is modified to - SELECT "id", "Themename","PrimaryColor", "BackgroundColor", "FontColor", "SecondaryColor" FROM "Themes" ORDER BY id LIMIT 10; the error is not seen.

The difference in the above two queries is that in the second query, BorderRadius column is not selected. So it looks like BorderRadius column is causing the issue

btsgh avatar May 06 '22 09:05 btsgh

@dilippitchika The issue is not related to theming because it is still happening when there is no BorderRadius column. This is related to actions and queries. Pulling the issue out from theming stability epic.

jsartisan avatar Jul 01 '22 05:07 jsartisan

@btsgh can you please check this or a similar scenario once and let me know if it's still occuring? We need to know more info and reproduce it in other apps

dilippitchika avatar Feb 16 '23 07:02 dilippitchika

@dilippitchika - The above habitasaurus application that is attached in this bug itself now does not function correctly as a lot of things in the UI have changed since then. I cannot seem to apply the theme at all

btsgh avatar Feb 21 '23 09:02 btsgh