appsmithorg / appsmithorg/appsmith

[Feature] Deep freeze properties that should not get updated before evaluation

オープン
#3,136 コメント 1 件 リアクション 1 件 担当者 0 名 GitHub で見る
Enhancement Javascript Product JS Evaluation Low Query & JS Pod Query Widgets & IDE Pod
主要言語
TypeScript
スター
40.9k
フォーク
4.8k
平均マージ
1日 22時間
マージ済み PR(30日)
45

説明

## Summary

Today I can add a binding like the following anywhere and cause data issues in my app.

```
Table1.tableData => {{ (()=>{ Api1.data = [{"hello": "world"}]; return Api1.data;})() }}
```

## Motivation

While the example above might look trivial, subtler versions like `.shift()` to get the first row (keys) from a google sheet can cause issues that would become really difficult for the user to debug later on.

Deep freezing will make sure that the user gets an error when they accidentally mutate an object that they're not supposed to.

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。