Skip to content

fix: include more information in the scripting error#8052

Merged
ihexxa merged 2 commits intodevelopfrom
fix/script-error-details
Oct 11, 2024
Merged

fix: include more information in the scripting error#8052
ihexxa merged 2 commits intodevelopfrom
fix/script-error-details

Conversation

@ihexxa
Copy link
Contributor

@ihexxa ihexxa commented Oct 8, 2024

Background

We tried to reproduce issue reported in #7976 and #8042 but no luck.
Stack information is included for identification.

Changes

  • Also include stack information if possible, for scripting errors
  • Fallback to empty array when iterationData is not defined

Ref: INS-4540

filfreire
filfreire previously approved these changes Oct 8, 2024
@ihexxa ihexxa self-assigned this Oct 9, 2024
@ihexxa ihexxa force-pushed the fix/script-error-details branch from 9d41c26 to 894888a Compare October 9, 2024 03:10
@ihexxa ihexxa force-pushed the fix/script-error-details branch from 894888a to 58fd1b6 Compare October 11, 2024 02:26
@ihexxa ihexxa enabled auto-merge (squash) October 11, 2024 02:26
@ihexxa ihexxa merged commit a7a9362 into develop Oct 11, 2024
@ihexxa ihexxa deleted the fix/script-error-details branch October 11, 2024 02:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants