Skip to content

Refresh workspaces when logging in and out #124

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Aug 18, 2023
Merged

Conversation

code-asher
Copy link
Member

Closes #112

Otherwise you get some confusing ECONNREFUSED errors in the logs.
@code-asher code-asher force-pushed the asher/refresh-workspaces branch from 952d3f5 to 4b80455 Compare August 18, 2023 07:36
Currently the refresh button does exactly nothing.

This will also let us load new workspaces in other cases, like when
logging in or out.
@code-asher code-asher force-pushed the asher/refresh-workspaces branch from 4b80455 to dd9d42f Compare August 18, 2023 07:38
@code-asher code-asher requested a review from kylecarbs August 18, 2023 07:40
@code-asher code-asher force-pushed the asher/refresh-workspaces branch 3 times, most recently from 176ef81 to 4ab3038 Compare August 18, 2023 08:09
I moved the async call out to make it easier to avoid adding new watches
if we log out while fetching workspaces.
@code-asher code-asher force-pushed the asher/refresh-workspaces branch from 4ab3038 to cb95d39 Compare August 18, 2023 09:00
@code-asher code-asher merged commit 4d599d2 into main Aug 18, 2023
@code-asher code-asher deleted the asher/refresh-workspaces branch August 18, 2023 17:16
@bpmct
Copy link
Member

bpmct commented Aug 18, 2023

Nice!!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Doesn't list workspaces in sidebar after initially authenticating
3 participants