[PR #13951] [MERGED] Reduce amount of telemetry events #29849

Open
opened 2026-01-31 09:37:14 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/13951
Author: @lhecker
Created: 9/8/2022
Status: ✅ Merged
Merged: 9/9/2022
Merged by: @undefined

Base: main ← Head: dev/lhecker/telemetry


📝 Commits (1)

  • ac18870 Reduce amount of telemetry events

📊 Changes

9 files changed (+65 additions, -107 deletions)

View changed files

📝 src/cascadia/TerminalApp/AppLogic.cpp (+35 -43)
📝 src/cascadia/TerminalApp/CommandPalette.cpp (+4 -4)
📝 src/cascadia/TerminalApp/TabManagement.cpp (+0 -27)
📝 src/cascadia/TerminalApp/TerminalPage.cpp (+17 -9)
📝 src/cascadia/TerminalConnection/ConptyConnection.cpp (+2 -2)
📝 src/cascadia/WindowsTerminal/AppHost.cpp (+0 -7)
📝 src/cascadia/WindowsTerminal/IslandWindow.cpp (+0 -1)
📝 src/cascadia/WindowsTerminal/main.cpp (+0 -6)
📝 src/inc/WilErrorReporting.h (+7 -8)

📄 Description

This commit reduces the amount of telemetry during general usage by about half.
8 events that weren't really used anymore were removed.
1 new event was added ("AppInitialized") which will help us investigate #5907.
During review 9 events were found that were incorrectly tagged as perf. data.

Validation Steps Performed

  • Launch Windows Terminal
  • The "latency" field "AppInitialized" matches the approx. launch time ✅

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/13951 **Author:** [@lhecker](https://github.com/lhecker) **Created:** 9/8/2022 **Status:** ✅ Merged **Merged:** 9/9/2022 **Merged by:** [@undefined](undefined) **Base:** `main` ← **Head:** `dev/lhecker/telemetry` --- ### 📝 Commits (1) - [`ac18870`](https://github.com/microsoft/terminal/commit/ac18870f7ecf6ea3f9873895189b543a84883d71) Reduce amount of telemetry events ### 📊 Changes **9 files changed** (+65 additions, -107 deletions) <details> <summary>View changed files</summary> 📝 `src/cascadia/TerminalApp/AppLogic.cpp` (+35 -43) 📝 `src/cascadia/TerminalApp/CommandPalette.cpp` (+4 -4) 📝 `src/cascadia/TerminalApp/TabManagement.cpp` (+0 -27) 📝 `src/cascadia/TerminalApp/TerminalPage.cpp` (+17 -9) 📝 `src/cascadia/TerminalConnection/ConptyConnection.cpp` (+2 -2) 📝 `src/cascadia/WindowsTerminal/AppHost.cpp` (+0 -7) 📝 `src/cascadia/WindowsTerminal/IslandWindow.cpp` (+0 -1) 📝 `src/cascadia/WindowsTerminal/main.cpp` (+0 -6) 📝 `src/inc/WilErrorReporting.h` (+7 -8) </details> ### 📄 Description This commit reduces the amount of telemetry during general usage by about half. 8 events that weren't really used anymore were removed. 1 new event was added ("AppInitialized") which will help us investigate #5907. During review 9 events were found that were incorrectly tagged as perf. data. ## Validation Steps Performed * Launch Windows Terminal * The "latency" field "AppInitialized" matches the approx. launch time ✅ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:37:14 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#29849