Skip to content

JAVAVSCODE-863: Notebook project context does not appear to be working on windows - #617

Merged
sid-srini merged 1 commit into
26.1.0from
JAVAVSCODE-863
Jul 22, 2026
Merged

JAVAVSCODE-863: Notebook project context does not appear to be working on windows#617
sid-srini merged 1 commit into
26.1.0from
JAVAVSCODE-863

Conversation

@singh-akhilesh

@singh-akhilesh singh-akhilesh commented Jun 23, 2026

Copy link
Copy Markdown
Member

Windows-only notebook project-context issue where an .ijnb file of the Java project unable to resolves the project context correctly in the dropdown/settings.

@oracle-contributor-agreement oracle-contributor-agreement Bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Jun 23, 2026
@singh-akhilesh
singh-akhilesh requested review from Achal1607 and sid-srini and removed request for Achal1607 June 23, 2026 13:21
@sid-srini
sid-srini changed the base branch from main to 26.0.1 June 23, 2026 13:38
@singh-akhilesh singh-akhilesh changed the title JAVAVSCODE-863 JAVAVSCODE-863: Notebook project context does not appear to be working on windows Jun 24, 2026
@sid-srini
sid-srini changed the base branch from 26.0.1 to main June 25, 2026 18:11
@sid-srini
sid-srini changed the base branch from main to 26.1.0 June 29, 2026 07:05
@sid-srini sid-srini added this to the JVSC 26.1.0 milestone Jun 29, 2026

@sid-srini sid-srini left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @singh-akhilesh for fixing these issues.

Please address the minor comment about localisation. Otherwise, LGTM 👍
Thanks

Comment thread vscode/src/commands/notebook.ts Outdated

const yes = l10n.value("jdk.extension.cache.label.confirmation.yes")
const cancel = l10n.value("jdk.extension.cache.label.confirmation.cancel")
const confirmation = await window.showWarningMessage("Project context changed. Do you want to restart the notebook kernel?",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please add this message in the l10n/bundle.l10n.*.json files with a message key and use l10n.value(<messageKey>) here. Thanks.

@sid-srini sid-srini left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @singh-akhilesh. LGTM 👍

@sid-srini
sid-srini merged commit de188b3 into 26.1.0 Jul 22, 2026
4 checks passed
@sid-srini
sid-srini deleted the JAVAVSCODE-863 branch July 22, 2026 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants