netsuite-suitecloud-sdk icon indicating copy to clipboard operation
netsuite-suitecloud-sdk copied to clipboard

Expose commands' status for commands

Open rajatWipfli opened this issue 1 year ago • 5 comments

Select the SuiteCloud tool where you encountered the issue

SuiteCloud Extension for VSCode

Version of the SuiteCloud Tool

v1.4.3

Operating system

Windows 10

Node Version / VSCode Version

v.18.15.0, V1.80

JAVA Version

17.x

Bug description

Is it possible to get the command's result?

I think if we return a value e.g. return action.run from https://github.com/oracle/netsuite-suitecloud-sdk/blob/fc85dfc0d5d3a5a9349cd8097ae5b24738cc9dc6/packages/vscode-extension/src/suitecloud.ts#L38-L45. It will expose that status.

I am building an internal extension for our project workflows which triggers SuiteClound extension commands. These commands are async but the resulting promise is completed immediately because we don't wait on the action to run.

I am calling executeCommand where docs say that it will return the value of the handler function. image

Steps To Reproduce

NA

Actual Output

No response

Expected Output

No response

Anything else?

No response

rajatWipfli avatar Jul 17 '23 13:07 rajatWipfli

As this is not causing any problem to the extension itself, I'll mark the issue as enhancement. Maybe we could be: return action.run(uri); on line 44 so your code could be awaiting for the resulting promise.

We'll take a look and evaluate.

davidecorreu avatar Jul 18 '23 14:07 davidecorreu

Yeah, thanks.

rajatWipfli avatar Jul 19 '23 07:07 rajatWipfli

@davidecorreu No rush but did you get the chance to review this? I am trying to understand the next steps. Do you have any estimates on when action would be taken on this?

rajatWipfli avatar Jul 28 '23 06:07 rajatWipfli

Any plan for this?

rajatWipfli avatar Dec 30 '23 12:12 rajatWipfli

Hi @rajatWipfli we will go over it and evaluate the fix.

tourajvaziri avatar Jan 02 '24 11:01 tourajvaziri