Skip to content

Move AzureConnection's strings into localizable resources#3463

Merged
DHowett-MSFT merged 4 commits intomasterfrom
dev/duhowett/localize_azure
Nov 7, 2019
Merged

Move AzureConnection's strings into localizable resources#3463
DHowett-MSFT merged 4 commits intomasterfrom
dev/duhowett/localize_azure

Conversation

@DHowett-MSFT
Copy link
Contributor

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.

@DHowett-MSFT DHowett-MSFT added the Area-AzureShell Workitems pertaining to the Azure Cloud Shell connection. label Nov 6, 2019
@DHowett-MSFT
Copy link
Contributor Author

Please don't comment on the ResX schema comment. It's not removable. VS just adds it back every single time you save.

@DHowett-MSFT
Copy link
Contributor Author

Yes, even with its spelling errors.

Copy link
Member

@zadjii-msft zadjii-msft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

its-all-coming-together

@DHowett-MSFT DHowett-MSFT merged commit d26865f into master Nov 7, 2019
@DHowett-MSFT DHowett-MSFT deleted the dev/duhowett/localize_azure branch November 7, 2019 20:26
@ghost
Copy link

ghost commented Nov 26, 2019

🎉Windows Terminal Preview v0.7.3291.0 has been released which incorporates this pull request.:tada:

Handy links:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-AzureShell Workitems pertaining to the Azure Cloud Shell connection.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants