Repository navigation
feat(data-warehouse): scaffold Razorpay source - #64484
Merged
Merged
Conversation
Contributor
|
Hey @Gilbert09! 👋 It looks like your git author email on this PR isn't your
You can fix it for this repo with: git config user.email "you@posthog.com"Or set it globally with |
👀 Auto-assigned reviewersThese soft owners were skipped because they only have minor changes here. Nothing blocks merge, so self-assign if you'd like a look:
Soft owners come from |
Contributor
|
Reviews (1): Last reviewed commit: "feat(data-warehouse): scaffold Razorpay ..." | Re-trigger Greptile |
Contributor
🕸️ Eager graphHow much code each root forces the browser to download and decode through static imports — the regression class total bundle size can't see.
✅ Largest files eagerly reachable from
|
| Size | File |
|---|---|
| 886.1 KiB | src/styles/global.scss |
| 609.0 KiB | public/hedgehog/burning-money-hog.png |
| 541.9 KiB | public/hedgehog/waving-hog.png |
| 448.2 KiB | public/hedgehog/stop-sign-hog.png |
| 362.0 KiB | public/hedgehog/phone-pair-hogs.png |
| 354.8 KiB | ../node_modules/.pnpm/@posthog+icons@0.36.6_react-dom@18.3.1_react@18.3.1__react@18.3.1/node_modules/@posthog/icons/dist/posthog-icons.es.js |
| 335.6 KiB | public/hedgehog/desk-hog.png |
| 329.6 KiB | src/taxonomy/core-filter-definitions-by-group.json |
| 323.2 KiB | public/hedgehog/3-bears-hogs.png |
| 292.5 KiB | src/lib/api.ts |
Largest files eagerly reachable from src/scenes/AuthenticatedShell.tsx
| Size | File |
|---|---|
| 886.1 KiB | src/styles/global.scss |
| 764.5 KiB | src/queries/validators.js |
| 609.0 KiB | public/hedgehog/burning-money-hog.png |
| 541.9 KiB | public/hedgehog/waving-hog.png |
| 448.2 KiB | public/hedgehog/stop-sign-hog.png |
| 362.0 KiB | public/hedgehog/phone-pair-hogs.png |
| 354.8 KiB | ../node_modules/.pnpm/@posthog+icons@0.36.6_react-dom@18.3.1_react@18.3.1__react@18.3.1/node_modules/@posthog/icons/dist/posthog-icons.es.js |
| 335.6 KiB | public/hedgehog/desk-hog.png |
| 329.6 KiB | src/taxonomy/core-filter-definitions-by-group.json |
| 323.2 KiB | public/hedgehog/3-bears-hogs.png |
Posted automatically by check-eager-graph · sizes are input-source bytes from the esbuild metafile · part of #32479
Contributor
|
Size Change: 0 B Total Size: 64.5 MB ℹ️ View Unchanged
|
Contributor
|
⏭️ Skipped snapshot commit because branch advanced to The new commit will trigger its own snapshot update workflow. If you expected this workflow to succeed: This can happen due to concurrent commits. To get a fresh workflow run, either:
|
Contributor
Migration SQL ChangesHey 👋, we've detected some migrations on this PR. Here's the SQL output for each migration, make sure they make sense:
|
Contributor
🔍 Migration Risk AnalysisWe've analyzed your migrations for potential risks. Summary: 0 Safe | 1 Needs Review | 0 Blocked
|
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
ghost
reviewed
Jun 18, 2026
ghost
left a comment
ghost
left a comment
There was a problem hiding this comment.
LGTM
Both previous issues (unregenerated schema.json and missing snapshot) are now fixed in cd0b692. The scaffolding follows existing patterns correctly across all 17 files — enum additions, migration, generated types, and the source stub are all consistent and complete.
What this PR does
Scaffolds a hidden Razorpay data warehouse source stub: adds the enum value, source config, generated configs, migration, icon, and all generated frontend/MCP type changes. No sync logic included. Previous review issues (missing schema regeneration and snapshot update) have been fixed.
Tag @mendral-app with feedback or questions. View session
danielcarletti
approved these changes
Jun 19, 2026
Gilbert09
force-pushed
the
posthog-code/scaffold-razorpay-source
branch
from
June 19, 2026 15:59
2cb8c81 to
7fb2901
Compare
Scaffolds a hidden stub for a Razorpay (payments) data warehouse source (unreleasedSource=True). Registers the source type, generated config, migration, and icon. No sync logic yet. Generated-By: PostHog Code Task-Id: f63fc515-6106-40e5-b75d-60e1997342fa
Regenerate the query schema JSON and the source-config generator snapshot to include the newly scaffolded Razorpay source. The Razorpay enum value was added to schema-general.ts but the derived schema.json was not rebuilt, which made schema.py regeneration drop the value and failed the snapshot test. Generated-By: PostHog Code Task-Id: 2566ae17-6ceb-46a3-aa81-2c68ba66af8a
Gilbert09
force-pushed
the
posthog-code/scaffold-razorpay-source
branch
from
June 19, 2026 16:10
f6de3e1 to
7d621dc
Compare
fuziontech
added a commit
that referenced
this pull request
Jun 19, 2026
…fill-admin * origin/master: (203 commits) fix(experiments): add missing event properties to the backend events on the recalculation workflow. (#64923) refactor(capture): decouple v1 RequestContext from analytics Context (#64184) feat(data-warehouse): add direct_query_enabled toggle to data sources (#64685) feat(data-warehouse): persist duckling bucket on DuckgresServer (#64919) fix(dev): strip read-denied $HOME dirs from sandbox PATH (#64768) fix(mcp-analytics): keep muted text readable inside quill subtrees (#64898) feat(skills): add Scouts tab backed by a generic skill category (#64796) fix(ducklake): make get_instance_region a lazy import in common helpers (#64913) fix(replay-vision): drop vestigial scanner_type from VisionAction selection (#64737) feat(warehouse): add xmin read path for postgres source (#64721) feat(warehouse): add per-table xmin sync radio (flag-gated) (#64718) feat(warehouse): validate xmin sync type and surface availability (#64723) fix(data-warehouse): CDC improvements — fix dead orphan-slot sweeper (#64725) feat(data-warehouse): scaffold Razorpay source (#64484) fix(dags): read duckling backfill bucket from stored catalog (#64909) feat(inbox): add archived tab with restore (#64876) feat(signals): surface a banner when inbox filters are active (#64811) fix(lemon-dialog): isolate nested form dialogs with unique kea logic keys (#62113) feat(session-replay): show event triggers on mobile replay settings tab (#64859) fix(preflight): restore kafka connectivity probe (#58743) ...
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
Razorpay (payment gateway) came up in user requests for data warehouse sources we don't yet support. This scaffolds the connector so it's registered and ready for a follow-up that implements sync.
Changes
Scaffolds a hidden stub for a Razorpay source following the
implementing-warehouse-sourcesskill:Razorpayvalue inExternalDataSourceType(backend enum +schema_enums.py+schema-general.tsfrontend list).RazorpaySourceregistered via_load_all.py, with an empty-fieldsSourceConfigin thePayments & billingcategory.RazorpaySourceConfigentry ingenerated_configs.py.source_typechoices onexternaldatasourceandpendingsourcecredential.frontend/public/services/razorpay.png.SOURCES.mdscaffolded-source list updated.The source ships with
unreleasedSource=True, so it's hidden from users. It has no sync logic — this is scaffolding only.How did you test this code?
I'm an agent. I ran
ruff checkandruff format --checkon the touched Python files (pass) andpy_compileon them (pass). Django wasn't available in my environment, so I derived the migration deterministically from the previoussource_typemigration (verified the diff is exactly the dependency bump plus the new choice tuple in bothAlterFieldblocks) and produced the generated-config and schema-enum entries by hand to match the generators' output. CI should be the source of truth formakemigrations --check,generate:source-configs, andschema:builddiffs.🤖 Agent context
Autonomy: Human-driven (agent-assisted)
Scaffolded with Claude Code in response to a request to add missing data warehouse sources surfaced from user survey submissions. Invoked the
/implementing-warehouse-sourcesand/django-migrationsskills. Razorpay was verified as a real, identifiable SaaS (payment gateway) with a Logo.dev brand before scaffolding. One PR per source; this PR is Razorpay only.