Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Bug] App crashes after adding a table widget when a button widget has a cyclic dependency #4209

Closed
akash-codemonk opened this issue Apr 29, 2021 · 0 comments · Fixed by #4214
Closed

[Bug] App crashes after adding a table widget when a button widget has a cyclic dependency #4209

akash-codemonk opened this issue Apr 29, 2021 · 0 comments · Fixed by #4214
Assignees
@SatishGandham
Labels
Bug Something isn't working Critical This issue needs immediate attention. Drop everything else Table Widget Verified When issue is retested post its fixed Widgets Product This label groups issues related to widgets

Comments

@akash-codemonk
Copy link
Contributor

Description

Steps to reproduce the behaviour:

  1. Add a button widget with. a cyclic dependency
  2. Add a Table widget the app crashes

Important Details

  • Version: [Cloud / Self-Hosted vx.x]
  • OS: [e.g.MacOSX]
  • Browser [e.g. chrome, safari]
tablecrash.mov
@akash-codemonk akash-codemonk added Bug Something isn't working Table Widget labels Apr 29, 2021
@akash-codemonk akash-codemonk mentioned this issue Apr 29, 2021
Closed
@github-actions github-actions bot added Widgets Product This label groups issues related to widgets UI Building Pod labels Apr 29, 2021
@Nikhil-Nandagopal Nikhil-Nandagopal added the Critical This issue needs immediate attention. Drop everything else label Apr 29, 2021
@Nikhil-Nandagopal Nikhil-Nandagopal removed their assignment Apr 29, 2021
SatishGandham pushed a commit that referenced this issue Apr 29, 2021
Fix cyclic dependency making the table widhget crash the app. #4209
9fa2fcd 
- Check if sanitizedTableData is array
@SatishGandham SatishGandham mentioned this issue Apr 29, 2021
Merged
7 tasks
sharat87 pushed a commit that referenced this issue May 4, 2021
@SatishGandham
Fix cyclic dependency making the table widhget crash the app. #4209 (#…
30b3333 
…4214)

- Check if sanitizedTableData is array

Co-authored-by: Satish Gandham <[email protected]>
@close-label close-label bot added the QA Needs QA attention label May 4, 2021
@prapullac prapullac added Verified When issue is retested post its fixed and removed QA Needs QA attention labels May 6, 2021
@coderabbitai coderabbitai bot mentioned this issue May 23, 2024
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SatishGandham SatishGandham

Labels
Bug Something isn't working Critical This issue needs immediate attention. Drop everything else Table Widget Verified When issue is retested post its fixed Widgets Product This label groups issues related to widgets
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix cyclic dependency making the table widget crash the app. #4209 appsmithorg/appsmith
6 participants
@SatishGandham @Nikhil-Nandagopal @akash-codemonk @vicky-primathon @prapullac @appsmith-bot