Installer: Removes unused telemetry functionality

[iOvergaard]

Summary

• Obsoletes the InstallHelper class and its SetInstallStatusAsync method
• Removes telemetry calls from installer that were no longer functioning (sending data to Our.Umbraco.com)
• Adds obsolete constructor overloads to maintain backward compatibility for DI
• Scheduled for removal in Umbraco 19

Background

Verified installer cookie usage all the way back to V8 - it was only ever used in this telemetry setting context. Removal approved in agreement with Sebastiaan.

Changes

• InstallHelper.cs: Gutted implementation, marked entire class as obsolete, converted SetInstallStatusAsync to no-op
• SettingsInstallController.cs: Removed SetInstallStatusAsync call, added obsolete constructor overload
• RegisterInstallCompleteStep.cs: Removed SetInstallStatusAsync call, added obsolete constructor overload

Test plan

• Verify installer completes successfully without telemetry calls
• Verify existing code using obsolete constructors still compiles with warnings
• Verify new code can use parameterless constructors

🤖 Generated with Claude Code

Co-Authored-By: Claude noreply@anthropic.com

[Copilot]

Pull request overview

This PR removes non-functional telemetry functionality from the Umbraco installer that was previously sending data to Our.Umbraco.com. The changes obsolete the InstallHelper class and its SetInstallStatusAsync method while maintaining backward compatibility through obsolete constructor overloads, all scheduled for removal in Umbraco 19.

Key Changes

• Gutted InstallHelper class implementation, removing all telemetry-related logic while keeping the public API surface for backward compatibility
• Removed calls to SetInstallStatusAsync from installer workflow (SettingsInstallController and RegisterInstallCompleteStep)
• Added obsolete constructor overloads to maintain DI backward compatibility

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File	Description
src/Umbraco.Infrastructure/Install/InstallHelper.cs	Marked entire class as obsolete, removed all field assignments and telemetry logic from constructor, converted SetInstallStatusAsync to a no-op method
src/Umbraco.Cms.Api.Management/Controllers/Install/SettingsInstallController.cs	Removed telemetry call from Settings endpoint, added obsolete constructor overload for backward compatibility with existing DI configurations
src/Umbraco.Infrastructure/Installer/Steps/RegisterInstallCompleteStep.cs	Removed InstallHelper field and telemetry call from Execute method, added obsolete constructor overload for backward compatibility

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

[AndyButland]

All looks good to me, thanks @iOvergaard. For good measure I tested the installer to make sure these updates had no negative effect on that functionality.