[PR #13437] [MERGED] Check for updates automatically (but don't install) #29549

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/13437
Author: @DHowett
Created: 7/5/2022
Status: ✅ Merged
Merged: 4/7/2023
Merged by: @DHowett

Base: main ← Head: dev/duhowett/check-for-updates


📝 Commits (10+)

  • 8381081 WIP: Check for updates when the about dialog is opened
  • f776327 Merge remote-tracking branch 'origin/main' into dev/duhowett/check-for-updates
  • d0d20d7 switch to uids
  • 2b9e3ef WIP: Move AboutDialog; part 1
  • ed41a39 Migrate spelling-0.0.21 changes from main
  • 587c72b Merge remote-tracking branch 'origin/main' into dev/duhowett/check-for-updates
  • 86d33a9 hey this is how we're actually supposed to do this
  • 5affca7 de-dupe code that got encapsulated
  • 5f35476 may as well add my nits while I'm here
  • c715a86 Fix compilation for non-dev

📊 Changes

11 files changed (+280 additions, -42 deletions)

View changed files

📝 .github/actions/spelling/expect/expect.txt (+2 -0)
➕ src/cascadia/TerminalApp/AboutDialog.cpp (+128 -0)
➕ src/cascadia/TerminalApp/AboutDialog.h (+40 -0)
➕ src/cascadia/TerminalApp/AboutDialog.idl (+19 -0)
➕ src/cascadia/TerminalApp/AboutDialog.xaml (+63 -0)
📝 src/cascadia/TerminalApp/Resources/en-US/Resources.resw (+12 -0)
📝 src/cascadia/TerminalApp/TerminalAppLib.vcxproj (+12 -0)
📝 src/cascadia/TerminalApp/TerminalPage.cpp (+1 -16)
📝 src/cascadia/TerminalApp/TerminalPage.h (+0 -2)
📝 src/cascadia/TerminalApp/TerminalPage.xaml (+2 -24)
📝 src/cascadia/TerminalApp/pch.h (+1 -0)

📄 Description

This PR adds support to the About Dialog for checking the store to see if there's a new version of the Terminal package available. We'll only do this once per day, per terminal window.

In dev mode, we'll always fake it and say there's an update to x.y.z available.

This also involved pulling all of the About dialog code out into its own class. All that is goodness.

We don't currently provide a button for installing the update. We just check. Incremental progress is better than none.


🔄 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/13437 **Author:** [@DHowett](https://github.com/DHowett) **Created:** 7/5/2022 **Status:** ✅ Merged **Merged:** 4/7/2023 **Merged by:** [@DHowett](https://github.com/DHowett) **Base:** `main` ← **Head:** `dev/duhowett/check-for-updates` --- ### 📝 Commits (10+) - [`8381081`](https://github.com/microsoft/terminal/commit/8381081c8e686538fc4c2765dff36333ad69861f) WIP: Check for updates when the about dialog is opened - [`f776327`](https://github.com/microsoft/terminal/commit/f7763278cb510f9d382808f5cdbb4bacfb66541d) Merge remote-tracking branch 'origin/main' into dev/duhowett/check-for-updates - [`d0d20d7`](https://github.com/microsoft/terminal/commit/d0d20d799ae0019a643cc246a64210238e88c01e) switch to uids - [`2b9e3ef`](https://github.com/microsoft/terminal/commit/2b9e3ef9c4d56e9e4f2906d7dcebecbd7e900d42) WIP: Move AboutDialog; part 1 - [`ed41a39`](https://github.com/microsoft/terminal/commit/ed41a39650d7ac18d7cebc28fd3903037804e347) Migrate spelling-0.0.21 changes from main - [`587c72b`](https://github.com/microsoft/terminal/commit/587c72b1a2de72f74a5900bcf48dc02e704a42e6) Merge remote-tracking branch 'origin/main' into dev/duhowett/check-for-updates - [`86d33a9`](https://github.com/microsoft/terminal/commit/86d33a970ae33586c45314751c2a4834e85426ac) hey this is how we're actually supposed to do this - [`5affca7`](https://github.com/microsoft/terminal/commit/5affca750439069e749acfedfe6f038e81294d1f) de-dupe code that got encapsulated - [`5f35476`](https://github.com/microsoft/terminal/commit/5f35476c5a065d3fde6f14754ef931064d49685c) may as well add my nits while I'm here - [`c715a86`](https://github.com/microsoft/terminal/commit/c715a86efb8f98855b6c3c717633c520011eaebc) Fix compilation for non-dev ### 📊 Changes **11 files changed** (+280 additions, -42 deletions) <details> <summary>View changed files</summary> 📝 `.github/actions/spelling/expect/expect.txt` (+2 -0) ➕ `src/cascadia/TerminalApp/AboutDialog.cpp` (+128 -0) ➕ `src/cascadia/TerminalApp/AboutDialog.h` (+40 -0) ➕ `src/cascadia/TerminalApp/AboutDialog.idl` (+19 -0) ➕ `src/cascadia/TerminalApp/AboutDialog.xaml` (+63 -0) 📝 `src/cascadia/TerminalApp/Resources/en-US/Resources.resw` (+12 -0) 📝 `src/cascadia/TerminalApp/TerminalAppLib.vcxproj` (+12 -0) 📝 `src/cascadia/TerminalApp/TerminalPage.cpp` (+1 -16) 📝 `src/cascadia/TerminalApp/TerminalPage.h` (+0 -2) 📝 `src/cascadia/TerminalApp/TerminalPage.xaml` (+2 -24) 📝 `src/cascadia/TerminalApp/pch.h` (+1 -0) </details> ### 📄 Description This PR adds support to the About Dialog for checking the store to see if there's a new version of the Terminal package available. We'll only do this once per day, per terminal window. In dev mode, we'll always fake it and say there's an update to `x.y.z` available. This also involved pulling all of the About dialog code out into its own class. All that is goodness. We don't currently provide a button for _installing_ the update. We just check. Incremental progress is better than none. --- <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:35:34 +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#29549