Conversation

andyleejordan

Refactors the logic a bit so the that Language Status Icon has the correct details when switching sessions. Also delete superfluous (and apparently broken) migration code.

@andyleejordan
Fix race condition with displaying PowerShell name on icon
3043a33
Refactors the logic a bit so the that Language Status Icon has the
correct details when switching sessions. Also delete superfluous (and
apparently broken) migration code.
@andyleejordanandyleejordan requested a review from a team August 11, 2023 18:23
SeeminglyScience
SeeminglyScience approved these changes Aug 11, 2023
View reviewed changes

Choose a reason for hiding this comment

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

LGTM!

@andyleejordanandyleejordan added this pull request to the merge queue Aug 12, 2023
@andyleejordanandyleejordan added Issue-BugA bug to squash.Area-UI labels Aug 12, 2023
@github-merge-queue-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 12, 2023
@andyleejordanandyleejordan added this pull request to the merge queue Aug 14, 2023
@github-merge-queue-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 14, 2023
@andyleejordanandyleejordan added this pull request to the merge queue Aug 14, 2023
Merged via the queue into main with commit 24d6dcb Aug 14, 2023
@andyleejordanandyleejordan deleted the andyleejordan/fix-status-icon branch August 14, 2023 17:15
Sign up for free to join this conversation on . Already have an account? Sign in to comment

@SeeminglyScience SeeminglyScience SeeminglyScience approved these changes

No one assigned
Area-UI Issue-BugA bug to squash.
Flying Fox
Status: Done
No milestone

Successfully merging this pull request may close these issues.