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

Download wallet #3546

Merged
merged 3 commits into from
Feb 8, 2022
Merged

Download wallet #3546

merged 3 commits into from
Feb 8, 2022

Conversation

jhorvath
Copy link
Contributor

@jhorvath jhorvath commented Feb 1, 2022

Download DB wallet action is available in vscode. NotifyDescriptor.InputLine is displayed in vscode as QuickInput. NotifyDescriptor.PasswordLine displays password input dialog.

Copy link

@JaroslavTulach JaroslavTulach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update dependency of java.lsp.server on openide.dialogs - it needs to use the most recent version otherwise there are goingto be linkage problems.

@@ -188,6 +188,23 @@
<specification-version>1.10</specification-version>
</run-dependency>
</dependency>
<dependency>
<code-name-base>org.openide.filesystems</code-name-base>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The cloud.oracle is using API from dialogs 7.59 - does it require version 7.59?

@@ -537,7 +537,7 @@
<build-prerequisite/>
<compile-dependency/>
<run-dependency>
<specification-version>7.52</specification-version>
<specification-version>7.59</specification-version>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good, thanks.

@@ -85,6 +86,9 @@ public static synchronized UIContext find() {
protected abstract CompletableFuture<MessageActionItem> showMessageRequest(ShowMessageRequestParams msg);
protected abstract void logMessage(MessageParams msg);
protected abstract Message showStatusMessage(ShowStatusMessageParams msg);
protected CompletableFuture<String> showInputBox(ShowInputBoxParams params) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, this is an API change in the API of java.lsp.server module. It would deserve module version increment, javadoc, @since tag. I am sorry for not documenting it in my commit that introduced this method.

@sdedic sdedic mentioned this pull request Feb 7, 2022
@sdedic
Copy link
Member

sdedic commented Feb 8, 2022

All green ! merging.

@sdedic sdedic merged commit 927a880 into apache:master Feb 8, 2022
@jhorvath jhorvath deleted the download-wallet branch March 7, 2022 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants