-
Notifications
You must be signed in to change notification settings - Fork 135
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
Dogfood recent changes #733
Merged
Merged
Conversation
This file contains 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
HostsSanitizer refactored out and shared
…Bridge RUMM-1785 DatadogEventBridge implemented
Optionals to non-optionals, non-optionals to optionals ashes to ashes, dust to dust...
WKUserContentController_DatadogTests improved
WebRUMEvents are decorated as raw JSONs instead of serializing them into RUMDataModels
Missing test case added: sampled out native session
RUMM-1791 WebEventConsumers added
RUMWebViewCommand is an empty command that keeps the session alive
…entCommand RUMM-1793 WebView events fire RUMWebViewCommand
Adds support for flutter error source (RUMM-1900)
hybrid-application: merge from master
It was in ns and that resulted in dropped events
…urce RUMM-1900/1904 Update RUM Schema
…event-date RUMM-1649 Fix: WebRUMEvent date offset in Ms
Problem was CR's `queue` leaking reference to global `userLogger` and running into threading race conditions on this being replaced by SDK's tear down logic in tests.
WebViewScenarioTest uses an actual website to test WebView Tracking feature
…-sdk-1.1.0 Update to version 1.1.0 of testing framework Update test targets so they keep the previous behaviour (Stdout and Stderr capturing must be explicitly activated now)
…ng test plan this is done by switching crash reporting `.xctestplan` to not include new tests automatically and removing Webviews test from the existing list.
…-apps-with-master RUMM-1649 Update `hybrid-application` branch with `master`
as this is now possible for Browser SDK views.
sent from Browser SDK.
- so logs can be properly digested and processed by BE.
…ot all events within our tests.
…tests RUMM-1649 WebViewTracking integration test added
…webview-logs-being-not-sent-properly RUMM-1924 Fix bug with WebView logs not appearing in Logs Explorer
to unify it with `dd-sdk-android` and with Browser SDK events.
…ile no view is active as this scenario is very likely to happen when tracking WebViews and it doesn't indicate user mistake.
…mages RUMM-1883 Report binary image with no UUID
…console-being-spammed-on-every-webview-event RUMM-1926 Do not print warning when receiving 'keep alive' command in WebView tracking
…ttribute-to-to-all-RUM-events RUMM-1925 Add `@service` attribute to all RUM events
…L_APIS` compiler flag to not expose it in releases until validated. It will be still available in dogfood projects.
…SDK events - now possible after DataDog/browser-sdk#1271
…ew-tracking-for-dogfooding RUMM-1911 Prepare WebView tracking for dogfooding
Issue: WebEventBridge.swift:44:80: variable declared in 'guard' condition is not usable in its body
…racking-compilation-for-Xcode-12 RUMM-1911 Fix compilation issue in nightly tests (macOS Catalina)
maxep
approved these changes
Jan 20, 2022
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.
🚀
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.
What and why?
🐶 Dogfooding recent changes (notably WebView tracking feature):
@service
attribute to all RUM events #725How?
First, merging
master
→dogfooding
. Then I'll contribute to dogfood projects with configuring WebView tracking for particular use cases.Review checklist