Skip to content

Source System Properties configuration - #1584

Merged
thoragudf merged 2 commits into
mainfrom
source-system-properties
Sep 8, 2026
Merged

thoragudf merged 2 commits into
mainfrom
source-system-properties

Conversation

@logason

@logason logason commented Nov 18, 2025 •

Copy link
Copy Markdown
Member

Note

Adds a new “System Properties” section to the source overview, documenting per-source configuration with Always/Sometimes/Never options.

  • Documentation (Sources > Overview):
    • New section: System Properties
      • Explains per-source configuration of system properties.
      • Defines options: Always, Sometimes, Never and their effects on Inspector validation and Codegen initialization.
      • Notes placement below the events table and provides an example (device_type on iOS).

Written by Cursor Bugbot for commit 52efc19. This will update automatically on new commits. Configure here.

@vercel

vercel Bot commented Nov 18, 2025 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Sep 8, 2026 2:01pm UTC

Request Review

@logason

logason commented Nov 18, 2025 •

Copy link
Copy Markdown
Member Author

@thoragudf thoragudf left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should we cross reference this in the property docs? That the system property presence is configurable om the source also. And in the other direction also, link to the property docs here

@logason

logason commented Nov 19, 2025

Copy link
Copy Markdown
Member Author

Should we cross reference this in the property docs? That the system property presence is configurable om the source also. And in the other direction also, link to the property docs here

@cursoragent please act on this feedback, the properties docs are here: https://www.avo.app/docs/data-design/avo-tracking-plan/properties#system-properties

@thoragudf

Copy link
Copy Markdown
Contributor

Addressed the cross-referencing feedback in 4a44293: the source overview's System Properties section now links to the properties docs, and the properties docs' System properties section has a new "Presence per source" note linking back to the source overview.

@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: f9b4c99d-872d-472d-b944-ccb4805ae83e


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

@thoragudf
thoragudf merged commit 761a275 into main Sep 8, 2026
4 checks passed
@thoragudf
thoragudf deleted the source-system-properties branch September 8, 2026 14:02

This branch was successfully deployed

1 active deployment
Preview — 4a442939 Deployed Sep 8, 2026 by vercel[bot]
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.

3 participants