Skip to content

feat: Add a walkthrough#321

Merged
Artmann merged 4 commits intomainfrom
chris/create-walkthrough
Feb 17, 2026
Merged

feat: Add a walkthrough#321
Artmann merged 4 commits intomainfrom
chris/create-walkthrough

Conversation

@Artmann
Copy link
Contributor

@Artmann Artmann commented Feb 17, 2026

CleanShot 2026-02-16 at 18 22 52

Fixes #317

Summary by CodeRabbit

  • New Features

    • Added an interactive "Deepnote Welcome" onboarding walkthrough with four guided steps: explore projects, work with notebook blocks, connect data sources, and set up a Python environment — each with step-specific instructions and visual aids.
  • Documentation

    • Added localized titles, descriptions, and image alt text for the walkthrough and its steps.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 17, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

A walkthrough manifest for Deepnote onboarding has been added to the extension. The package.json defines a new walkthrough with id "deepnoteWelcome" containing four sequential steps: exploring projects, working with notebook blocks, connecting data sources, and setting up a Python environment. Each step includes media assets and completion events. Corresponding localization strings are added to package.nls.json to support the walkthrough titles, descriptions, and media alt-text across the four steps.

🚥 Pre-merge checks | ✅ 5 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Updates Docs ❓ Inconclusive PR implements walkthrough feature but no documentation updates are visible in tracked changes. Verify documentation updates in deepnote/deepnote OSS and deepnote/deepnote-internal repositories.
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat: Add a walkthrough' directly describes the main change—adding a walkthrough feature to the extension.
Linked Issues check ✅ Passed The PR implements the objective from #317: a walkthrough guiding new users through Deepnote setup, covering project exploration, notebook blocks, data sources, and Python environment configuration.
Out of Scope Changes check ✅ Passed All changes are scoped to adding the walkthrough feature: package.json defines the walkthrough structure and steps, package.nls.json provides localization strings. No extraneous modifications detected.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link

codecov bot commented Feb 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0%. Comparing base (0f9dab9) to head (1ccaef2).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@     Coverage Diff     @@
##   main   #321   +/-   ##
===========================
===========================
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@package.nls.json`:
- Around line 298-317: This JSON file failed Prettier formatting; run Prettier
(or your repo's formatter) on package.nls.json and commit the reformatted file
so keys like
"contributes.walkthroughs.deepnoteWelcome.steps.exploreProjects.description" and
other "contributes.walkthroughs.deepnoteWelcome.*" entries are properly
formatted/escaped according to the project's Prettier config; ensure no trailing
commas, correct indentation, and valid JSON syntax before committing.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@package.nls.json`:
- Around line 301-305: The walkthrough references a non-existent command
deepnoteExplorer.focus in the package.nls.json key
contributes.walkthroughs.deepnoteWelcome.steps.exploreProjects.description;
update this to a registered command or register the command: either change the
command token to an existing exported/declared command such as
deepnote.refreshExplorer or deepnote.revealInExplorer (ensuring the token
matches an entry in package.json "contributes.commands" and the command
implementation name), or add a new command named deepnoteExplorer.focus to
package.json and implement/exports that command in the extension activation code
so the walkthrough link resolves.

@Artmann Artmann marked this pull request as ready for review February 17, 2026 09:27
@Artmann Artmann requested a review from a team as a code owner February 17, 2026 09:27
coderabbitai[bot]
coderabbitai bot previously approved these changes Feb 17, 2026
Copy link
Contributor

@dinohamzic dinohamzic left a comment

Choose a reason for hiding this comment

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

Nice improvement, left some suggestions.

package.nls.json Outdated
"contributes.walkthroughs.deepnoteWelcome.description": "Your first steps to set up and explore Deepnote notebooks in VS Code.",
"contributes.walkthroughs.deepnoteWelcome.steps.exploreProjects.title": "Explore Your Projects",
"contributes.walkthroughs.deepnoteWelcome.steps.exploreProjects.description": {
"message": "Open the Deepnote sidebar to browse your projects and notebooks. Create new projects, import existing notebooks, and organize your work.\n[Open Deepnote Explorer](command:deepnoteExplorer.focus)",
Copy link
Contributor

Choose a reason for hiding this comment

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

This can be simplified to:

Use the Deepnote sidebar to browse, create, import, and organize your projects and notebooks.

package.nls.json Outdated
"comment": ["{Locked='command:deepnoteExplorer.focus'}"]
},
"contributes.walkthroughs.deepnoteWelcome.steps.exploreProjects.media.altText": "The Deepnote Explorer sidebar showing projects and notebooks",
"contributes.walkthroughs.deepnoteWelcome.steps.notebookBlocks.title": "Work with Notebook Blocks",
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we be more explicit here? It's unclear what "work" really means here.

eg. "Create your first notebook block"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point.

"contributes.walkthroughs.deepnoteWelcome.steps.connectDataSources.title": "Connect to Your Data Sources",
"contributes.walkthroughs.deepnoteWelcome.steps.connectDataSources.description": "Set up integrations to connect your notebooks to databases, data warehouses, and other services. Query your data directly from SQL blocks.",
"contributes.walkthroughs.deepnoteWelcome.steps.connectDataSources.media.altText": "The Manage Integrations panel for connecting data sources",
"contributes.walkthroughs.deepnoteWelcome.steps.setupEnvironment.title": "Set Up a Python Environment",
Copy link
Contributor

Choose a reason for hiding this comment

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

Should setting up a python environment be the first thing the user does?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe, but I think it's less important for showing the value of the extension.

@Artmann Artmann merged commit 6e4b391 into main Feb 17, 2026
13 checks passed
@Artmann Artmann deleted the chris/create-walkthrough branch February 17, 2026 10:59
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.

Create a walkthrough that explains how to get started

2 participants