Skip to content

Conversation

@nh758
Copy link
Contributor

@nh758 nh758 commented Feb 19, 2025

Release Notes

  • Add support for NetSuite API objects

Test Status

nh758 and others added 25 commits September 9, 2024 13:06
@nh758 nh758 added the minor Tag Pull Requests to trigger a minor version update label Feb 19, 2025

_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

Unused variable fieldTypes.

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.

Suggested changeset 1
src/rootPages/Designer/ui_work_object_list_newObject_netsuite_connections.js

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/src/rootPages/Designer/ui_work_object_list_newObject_netsuite_connections.js b/src/rootPages/Designer/ui_work_object_list_newObject_netsuite_connections.js
--- a/src/rootPages/Designer/ui_work_object_list_newObject_netsuite_connections.js
+++ b/src/rootPages/Designer/ui_work_object_list_newObject_netsuite_connections.js
@@ -222,3 +222,2 @@
          const self = this;
-         const fieldTypes = this.AB.Class.ABFieldManager.allFields();
          const fieldKeys = ["string", "LongText", "number", "date", "boolean"];
EOF
@@ -222,3 +222,2 @@
const self = this;
const fieldTypes = this.AB.Class.ABFieldManager.allFields();
const fieldKeys = ["string", "LongText", "number", "date", "boolean"];
Copilot is powered by AI and may make mistakes. Always verify output.
_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

Unused variable fieldKeys.

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.

Suggested changeset 1
src/rootPages/Designer/ui_work_object_list_newObject_netsuite_connections.js

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/src/rootPages/Designer/ui_work_object_list_newObject_netsuite_connections.js b/src/rootPages/Designer/ui_work_object_list_newObject_netsuite_connections.js
--- a/src/rootPages/Designer/ui_work_object_list_newObject_netsuite_connections.js
+++ b/src/rootPages/Designer/ui_work_object_list_newObject_netsuite_connections.js
@@ -223,3 +223,2 @@
          const fieldTypes = this.AB.Class.ABFieldManager.allFields();
-         const fieldKeys = ["string", "LongText", "number", "date", "boolean"];
 
EOF
@@ -223,3 +223,2 @@
const fieldTypes = this.AB.Class.ABFieldManager.allFields();
const fieldKeys = ["string", "LongText", "number", "date", "boolean"];

Copilot is powered by AI and may make mistakes. Always verify output.
@nh758 nh758 merged commit ef15f95 into master Feb 19, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor Tag Pull Requests to trigger a minor version update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants