[PR #3463] [MERGED] Move AzureConnection's strings into localizable resources #25368

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/3463
Author: @DHowett-MSFT
Created: 11/6/2019
Status: Merged
Merged: 11/7/2019
Merged by: @DHowett-MSFT

Base: masterHead: dev/duhowett/localize_azure


📝 Commits (4)

  • 234992b Move the AzureConnection's strings into localizable resources
  • fe1cf65 Merge remote-tracking branch 'github/master' into dev/duhowett/localize_azure
  • 9276c51 eh.
  • 898b0a0 Merge remote-tracking branch 'github/master' into dev/duhowett/localize_azure

📊 Changes

7 files changed (+296 additions, -90 deletions)

View changed files

📝 src/cascadia/TerminalConnection/AzureConnection.cpp (+68 -55)
📝 src/cascadia/TerminalConnection/AzureConnection.h (+1 -0)
src/cascadia/TerminalConnection/AzureConnectionStrings.h (+0 -32)
src/cascadia/TerminalConnection/Resources/en-US/Resources.resw (+208 -0)
📝 src/cascadia/TerminalConnection/TerminalConnection.vcxproj (+7 -1)
📝 src/cascadia/TerminalConnection/TerminalConnection.vcxproj.filters (+9 -2)
📝 src/cascadia/TerminalConnection/init.cpp (+3 -0)

📄 Description

Summary of the Pull Request

This also sets up TerminalConnection to have resources, which will be useful for the messaged in #3461.

This pull request does not adjust any of the state machine logic, but a following one will.

References

This was untracked work that was mentioned in passing in #2146 .

PR Checklist

  • Closes
  • CLA signed
  • Tests added/passed/manually tested
  • Requires documentation to be updated
  • Core

Validation Steps Performed

Walked through the Azure connection myself.


🔄 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/3463 **Author:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Created:** 11/6/2019 **Status:** ✅ Merged **Merged:** 11/7/2019 **Merged by:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Base:** `master` ← **Head:** `dev/duhowett/localize_azure` --- ### 📝 Commits (4) - [`234992b`](https://github.com/microsoft/terminal/commit/234992b50903494b23d3339e4c76c2535765f532) Move the AzureConnection's strings into localizable resources - [`fe1cf65`](https://github.com/microsoft/terminal/commit/fe1cf65b54431ce28e07eef5b79a64dbd56e8bd3) Merge remote-tracking branch 'github/master' into dev/duhowett/localize_azure - [`9276c51`](https://github.com/microsoft/terminal/commit/9276c5181d775c666ce8e2b5d80ebc7519f53fa2) eh. - [`898b0a0`](https://github.com/microsoft/terminal/commit/898b0a0751b2ae06ba7e3acb7b54e5f079f33e68) Merge remote-tracking branch 'github/master' into dev/duhowett/localize_azure ### 📊 Changes **7 files changed** (+296 additions, -90 deletions) <details> <summary>View changed files</summary> 📝 `src/cascadia/TerminalConnection/AzureConnection.cpp` (+68 -55) 📝 `src/cascadia/TerminalConnection/AzureConnection.h` (+1 -0) ➖ `src/cascadia/TerminalConnection/AzureConnectionStrings.h` (+0 -32) ➕ `src/cascadia/TerminalConnection/Resources/en-US/Resources.resw` (+208 -0) 📝 `src/cascadia/TerminalConnection/TerminalConnection.vcxproj` (+7 -1) 📝 `src/cascadia/TerminalConnection/TerminalConnection.vcxproj.filters` (+9 -2) 📝 `src/cascadia/TerminalConnection/init.cpp` (+3 -0) </details> ### 📄 Description ## Summary of the Pull Request This also sets up TerminalConnection to _have_ resources, which will be useful for the messaged in #3461. **This pull request does not adjust any of the state machine logic, but a following one will.** ## References This was untracked work that was mentioned in passing in #2146 . ## PR Checklist * [ ] Closes <nothing> * [x] CLA signed * [x] Tests added/passed/manually tested * [ ] Requires documentation to be updated * [x] Core ## Validation Steps Performed Walked through the Azure connection myself. --- <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:09:05 +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#25368