You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@daffodil.apache.org by "shanedell (via GitHub)" <gi...@apache.org> on 2023/06/16 16:40:15 UTC

[GitHub] [daffodil-vscode] shanedell commented on a diff in pull request #664: Launch Wizard Validation

shanedell commented on code in PR #664:
URL: https://github.com/apache/daffodil-vscode/pull/664#discussion_r1232489849


##########
src/launchWizard/launchWizard.js:
##########
@@ -18,12 +18,62 @@
 // Retrieve vscode api - Doing this multiple times causes issues with the scripts

Review Comment:
   So the reason there is both `launchWizard` .ts and .js is because the webview UI will directly communicate to JS scripts. However, we may need some data that only be gotten through the typescript, such as different npm packages. So the posting message stuff done in both JS and TS is how we communicate from the WebView JS to the TypeScript code.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@daffodil.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org