feat(experimentation): Snowflake warehouse connection support#7565
Conversation
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.
Tip: disable this comment in your organization's Code Review settings.
|
The latest updates on your projects. Learn more about Vercel for GitHub. 3 Skipped Deployments
|
Docker builds report
|
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7565 +/- ##
========================================
Coverage 98.48% 98.49%
========================================
Files 1415 1417 +2
Lines 53595 53749 +154
========================================
+ Hits 52785 52939 +154
Misses 810 810 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Visual Regression19 screenshots compared. See report for details. |
Thanks for submitting a PR! Please check the boxes below:
docs/if required so people know about the feature.Changes
configJSONField (nullable) andcreatedstatus toWarehouseConnectionmodel, enabling type-specific configuration storageSnowflakeConfigTypedDict inexperimentation/types.pywith defaults for all fields exceptaccount_identifier(required)name, and partial update supportUpdateModelMixinto the ViewSet for PATCH support with audit loggingHow did you test this code?