docs: clarify self-hosted analytics event shape#45256
docs: clarify self-hosted analytics event shape#45256princepal9120 wants to merge 3 commits intosupabase:masterfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
7 Skipped Deployments
|
|
@princepal9120 is attempting to deploy a commit to the Supabase Team on Vercel. A member of the Team first needs to authorize it. |
|
Thanks for contributing to Supabase! ❤️ Our team will review your PR. A few tips for a smoother review process:
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughDocumentation clarifies Vector-based log ingestion for self-hosted analytics: fixes a typo and replaces vague schema guidance with concrete required event fields ( Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@princepal9120 Can you run |
|
@ChrisChinchilla thanks for the review. I fixed the failing format check by running Prettier on |
|
@ChrisChinchilla @Ziinc the PR is showing GitHub's merge warning: "Head branch was modified. Review and try the merge again." The approved commit was
Current state from my side:
Could you please re-check/re-approve the latest head or try the merge again when you get a chance? |
Summary
vector.ymlfor built-in Studio Logs panelsTests
Fixes #40926
Summary by CodeRabbit