-
Notifications
You must be signed in to change notification settings - Fork 2
Add support for NetSuite API objects #288
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.
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
Conversation
… Netsuite objects.
Team node content
|
|
||
| _fieldItem(key, type) { | ||
| const self = this; | ||
| const fieldTypes = this.AB.Class.ABFieldManager.allFields(); |
Check notice
Code scanning / CodeQL
Unused variable, import, function or class Note
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI 12 months ago
To fix the problem, we need to remove the unused variable fieldTypes from the function _fieldItem. This will improve code readability and eliminate any unnecessary memory usage. The change should be made in the file src/rootPages/Designer/ui_work_object_list_newObject_netsuite_connections.js on line 223.
| @@ -222,3 +222,2 @@ | ||
| const self = this; | ||
| const fieldTypes = this.AB.Class.ABFieldManager.allFields(); | ||
| const fieldKeys = ["string", "LongText", "number", "date", "boolean"]; |
| _fieldItem(key, type) { | ||
| const self = this; | ||
| const fieldTypes = this.AB.Class.ABFieldManager.allFields(); | ||
| const fieldKeys = ["string", "LongText", "number", "date", "boolean"]; |
Check notice
Code scanning / CodeQL
Unused variable, import, function or class Note
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI 12 months ago
To fix the problem, we need to remove the unused variable fieldKeys from the code. This will improve code readability and eliminate any unnecessary memory usage. The change should be made in the file src/rootPages/Designer/ui_work_object_list_newObject_netsuite_connections.js on line 224.
| @@ -223,3 +223,2 @@ | ||
| const fieldTypes = this.AB.Class.ABFieldManager.allFields(); | ||
| const fieldKeys = ["string", "LongText", "number", "date", "boolean"]; | ||
|
|
Release Notes
Test Status