Skip to content

docs: refresh core database integration screenshots - #476

Draft
petrfiedler wants to merge 1 commit into
mainfrom
docs-audit/core-database-integrations
Draft

petrfiedler wants to merge 1 commit into
mainfrom
docs-audit/core-database-integrations

Conversation

@petrfiedler

@petrfiedler petrfiedler commented Aug 19, 2026 •

Copy link
Copy Markdown
Contributor

What this changes

This draft refreshes the current Deepnote UI screenshots for the predefined core-database-integrations batch:

  • PostgreSQL setup and successful query state
  • MySQL setup
  • MongoDB setup
  • Supabase PostgreSQL setup and successful query state

The SQL Server and InfluxDB pages were also audited through live workflows, but no existing static asset was replaced for those pages because their current evidence does not correspond to the documented static screenshot action.

How this was verified

  • Used the real Google Chrome UI and only the approved Sample Workspace.
  • Restored temporary TCP tunnels to disposable local PostgreSQL, MySQL, MongoDB, SQL Server, and InfluxDB v2 fixtures.
  • Verified connection forms, schema browsing, SQL blocks, Python client execution, filtered query results, and DataFrame output where documented.
  • Captured replacement screenshots from the exact Chrome window, removed browser chrome and cursors, verified native dimensions, and encoded each replacement directly to WebP under the repository byte limit.
  • Ran validate_pr_diff.py against origin/main.
  • Ran pnpm test: 2,894 tests passed and 1 test was skipped.
  • Ran pnpm typecheck, pnpm prettier:check, and pnpm biome:check. Biome reported six pre-existing warnings in example files and a broken symlink; no fixes were applied.

Unverified by human

This draft uses the user-authorized incomplete-verification exception. The following exact gaps remain:

  • Supabase: Supabase account and project creation; provider-issued database host, database, port, user, password; current Supabase provider screenshot R1viC2QvqEjetTO9FLzw.webp. The Deepnote PostgreSQL workflow was verified with disposable local data, but no Supabase account was available and the audit does not handle real credentials.
  • InfluxDB: External Deepnote Templates workspace template duplication step; current external InfluxDB provider token-creation UI and screenshot; evidence did not verify the linked template workspace because the audit contract permits only Sample Workspace. The documented Python client and query path were verified with a disposable local InfluxDB v2 fixture.
  • PostgreSQL, MySQL, and SQL Server: separate SSL and SSH connection behavior was not independently exercised because the disposable fixtures intentionally used plain TCP and did not provide SSH servers. The current setup controls were observed in the live forms.
  • MongoDB: a separate secure transport or SSH connection was not independently exercised; the documented connection-string and Python query path were verified with disposable data.

No real provider account, password, API token, or private credential was handled. The replacement images use safe disposable test data or current Deepnote UI state only.

@petrfiedler petrfiedler added the documentation Improvements or additions to documentation label Aug 19, 2026
@coderabbitai

coderabbitai Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

💤 Files selected but had no reviewable changes (6)
  • assets/docs/GtjwnTvWTlVWEXTZwtCr.webp
  • assets/docs/KMoRPKCNQamVvyoUJIJx.webp
  • assets/docs/LvtKgFIQTKdtwhKqWrRg.webp
  • assets/docs/daU32gzMSFOGQbHw9EL3.webp
  • assets/docs/emfKwFxR5G9HoE6XNCt6.webp
  • assets/docs/xJmtNalsRLKv245XcPkD.webp
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: df1169d6-3aa2-4afa-bb60-faed1b2cb364

📥 Commits

Reviewing files that changed from the base of the PR and between a9b3cc0 and aa722cf.

📒 Files selected for processing (6)
  • assets/docs/GtjwnTvWTlVWEXTZwtCr.webp
  • assets/docs/KMoRPKCNQamVvyoUJIJx.webp
  • assets/docs/LvtKgFIQTKdtwhKqWrRg.webp
  • assets/docs/daU32gzMSFOGQbHw9EL3.webp
  • assets/docs/emfKwFxR5G9HoE6XNCt6.webp
  • assets/docs/xJmtNalsRLKv245XcPkD.webp

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

@codecov

codecov Bot commented Aug 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.30%. Comparing base (a9b3cc0) to head (aa722cf).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #476   +/-   ##
=======================================
  Coverage   88.30%   88.30%           
=======================================
  Files         191      191           
  Lines       10697    10697           
  Branches     3079     3079           
=======================================
  Hits         9446     9446           
  Misses       1249     1249           
  Partials        2        2           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@petrfiedler
petrfiedler requested a review from Haleshot August 19, 2026 13:52

@Haleshot Haleshot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM ✅

@Haleshot

Haleshot commented Aug 28, 2026 •

Copy link
Copy Markdown
Collaborator

@petrfiedler
Curious on whether pinggy was connected to your agent to capture these screenshots (that's what I gathered from the username fields from the updated screenshots).

@petrfiedler

Copy link
Copy Markdown
Contributor Author

@petrfiedler Curious on whether pinggy connected to your agent to capture these screenshots (that's what I gathered from the username fields from the updated screenshots).

My agent used Pinggy as a temporary public TCP tunnel for the PostgreSQL.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Needs supabase credentials. Use the sample values in the old screenshot but in the new UI.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Refresh this screenshot to be more representative for the supabase query - use the deleted one's content.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants