Skip to content
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

[vscode][proposed] documentPaste proposed API updates for 1.79 #12633

Closed
Tracked by #12613
rschnekenbu opened this issue Jun 21, 2023 · 2 comments · Fixed by #12737
Closed
Tracked by #12613

[vscode][proposed] documentPaste proposed API updates for 1.79 #12633

rschnekenbu opened this issue Jun 21, 2023 · 2 comments · Fixed by #12737
Assignees
Labels
builtins Issues related to VS Code builtin extensions vscode issues related to VSCode compatibility

Comments

@rschnekenbu
Copy link
Contributor

documentPaste proposed API has evolved between 1.78 and 1.79:

  • DocumentPasteEdit now has several required properties: id, label and optional priority. Constructor has also evolved to reflect these changes.
  • DocumentPasteProviderMetadata has a new optional Property (copyMimeTypes)
  • DocumentPasteEditProvider#providerDocumentPasteEdits() is now optional.

This proposed API is used by several builtins (ipynb & markdown-language-feature)

@vince-fugnitto vince-fugnitto added vscode issues related to VSCode compatibility builtins Issues related to VS Code builtin extensions labels Jun 21, 2023
@msujew msujew self-assigned this Jul 11, 2023
@JonasHelming
Copy link
Contributor

@msujew When do you plan to work on this? Would be nice to have this and #12632
for the upcoming community release

@msujew
Copy link
Member

msujew commented Jul 19, 2023

@JonasHelming I should have some time to work on both of these issues this week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
builtins Issues related to VS Code builtin extensions vscode issues related to VSCode compatibility
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants