-
Notifications
You must be signed in to change notification settings - Fork 0
Sync filter docs with the canonical envelope and cut 1.2.0 #34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -405,8 +405,13 @@ profiles.command('search', { | |
| 'apps.balance (+ "app_id", optional "chain_id"); ' + | ||
| 'tokens.balance (+ "token_address", "scope":"any"|"protocol", "app_id" when scope is "protocol", optional "chain_id"); ' + | ||
| 'labels.value (+ "tag_id", optional "chain_id"). ' + | ||
| 'op: eq, neq, gt, gte, lt, lte, in, nin, contains, notEmpty, isEmpty ' + | ||
| '(contains = substring, social fields only; notEmpty/isEmpty = value-less existence checks on string fields). ' + | ||
| 'op: eq, neq, gt, gte, lt, lte, in, nin, contains, startsWith, endsWith, notEmpty, isEmpty. ' + | ||
| 'Operator support is per field: the .balance fields take comparison operators only; ' + | ||
| 'contains works on routable string attributes (users.device, users.os, users.referrer, users.utm_*, users.click_id — case-sensitive), ' + | ||
| 'on social fields and on labels.value (case-insensitive); ' + | ||
| 'startsWith/endsWith are routable string attributes only; ' + | ||
| 'notEmpty/isEmpty are value-less existence checks on string fields; ' + | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
When users consult Useful? React with 👍 / 👎. |
||
| 'users.lifecycle takes only eq and in. ' + | ||
| 'Retired and rejected with a 400: identifier-in-path fields (chains.1.balance, apps.uniswap-v3.balance, tokens.0x….balance, labels.vip), ' + | ||
| 'the "appId" spelling, and long-form operators (equals, notEquals, greater, greaterOrEqual, less, lessOrEqual, notIn, includes).', | ||
| ), | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When a user follows this guidance for a resource predicate, such as
--filters '[{"field":"chains.balance","op":"gt","value":1,"chain_id":"1"}]', the command fails locally instead of reaching the API:validateAnalyticsFilteronly permitsfield,op,value, and optionalfilters(src/commands/analytics.ts:61-82), so every named qualifier listed here is rejected as an extra key. Either teach the analytics validator to accept and validate these qualifiers or avoid documenting them as usable through this CLI.Useful? React with 👍 / 👎.