Skip to content

Conversation

@andyleejordan
Copy link
Member

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.

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 andyleejordan requested a review from a team August 11, 2023 18:23
Copy link
Collaborator

@SeeminglyScience SeeminglyScience left a comment

Choose a reason for hiding this comment

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

LGTM!

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

Labels

Area-UI Issue-Bug A bug to squash.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants