Add Import Files... command that supports many file types#1764
Add Import Files... command that supports many file types#1764isc-bsaviano wants to merge 1 commit intointersystems-community:masterfrom
Import Files... command that supports many file types#1764Conversation
isc-klu
left a comment
There was a problem hiding this comment.
Why aren’t we concerned about key bindings for other command IDs? Do they simply not appear in the telemetry data?
|
We capture no telemetry data on user keybindings. It should be possible to add. |
|
That helps, thanks. I’m curious why we were comfortable deleting |
|
@isc-klu We have telemetry on command execution. Over the past 90 days, |
This PR fixes #1763 by removing restrictions on the existing
Import XML Files...command. The underlying server APIs support many file types, so it's easy for VS Code to do so as well. This also make theImport Local Files...command redundant, so I removed it. I kept thevscode-objectscript.importXMLFilescommand ID in case anyone made a keybinding for it, but I am also open to changing it to something likevscode-objectscript.importFiles.