Skip to content
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

feat: Support disabling INV, custom INV values, and FBC for Flutter #2212

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

fuzzybinary
Copy link
Member

What and why?

This adds an internal method - setInternalViewAttribute - which modifies a specific attribute collection for views. This attribute collection is checked for custom INV Values and Flutter First Build Complete metrics.

This also adds the ability to disable INV by setting its lastActionPredicate to nil, which has the factory return a nil INVMetricTracking object.

refs: RUM-8548

How?

A brief description of implementation details of this PR.

Review checklist

  • Feature or bugfix MUST have appropriate tests (unit, integration)
  • Make sure each commit and the PR mention the Issue number or JIRA reference
  • Add CHANGELOG entry for user facing changes
  • Add Objective-C interface for public APIs (see our guidelines [internal]) and run make api-surface)

fuzzybinary and others added 7 commits February 25, 2025 08:58
This adds an internal method - `setInternalViewAttribute` - which modifies a specific attribute collection for views. This attribute collection is checked for custom INV Values (if INV is disabled) and Flutter First Build Complete metrics.

This also adds the ability to disable INV by setting its last action predicate to `false`

refs: RUM-8548
Co-authored-by: Maciek Grzybowski <[email protected]>
# Conflicts:
#	DatadogRUM/Tests/RUMMonitor/Scopes/RUMViewScopeTests.swift
@fuzzybinary fuzzybinary requested review from a team as code owners February 28, 2025 17:07
@fuzzybinary fuzzybinary force-pushed the jward/RUM-8546-first-build-complete branch from 0626367 to 3df99d1 Compare February 28, 2025 17:10
@datadog-datadog-prod-us1
Copy link

Datadog Report

Branch report: jward/RUM-8546-first-build-complete
Commit report: 3df99d1
Test service: dd-sdk-ios

✅ 0 Failed, 742 Passed, 3090 Skipped, 50.19s Total duration (1m 45.94s time saved)

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.

1 participant