Skip to content

refactor: poll for git auth updates when creating a workspace #9804

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.

Sign up for GitHub

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 13 commits into from
Sep 26, 2023
Merged

refactor: poll for git auth updates when creating a workspace #9804

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
e199cbb
chore: xstate layout
aslilac Sep 20, 2023
ed72bf5
refactor: poll for git auth updates when connecting a provider
aslilac Sep 21, 2023
e0f72ef
🧹
aslilac Sep 22, 2023
cd75e5c
quit polling when authenticated
aslilac Sep 22, 2023
e4d67b9
remove `BroadcastChannel` stuff
aslilac Sep 22, 2023
8669a5f
🧹
aslilac Sep 22, 2023
479a790
fix tests
aslilac Sep 22, 2023
5475497
Merge branch 'main' into git-auth-polling
aslilac Sep 22, 2023
91665c6
fix tests pt. 2
aslilac Sep 22, 2023
f5d0b7e
tweak `versionId` logic a touch
aslilac Sep 25, 2023
18f8f6a
Merge branch 'main' into git-auth-polling
aslilac Sep 25, 2023
33874fb
add a test
aslilac Sep 26, 2023
e81800e
oh
aslilac Sep 26, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
quit polling when authenticated
  • Loading branch information
@aslilac
aslilac committed Sep 22, 2023
commit cd75e5c0aabd5e2eed2a4969b96de95bc1e5114c
35 changes: 21 additions & 14 deletions site/src/pages/CreateWorkspacePage/CreateWorkspacePage.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,27 @@ const CreateWorkspacePage: FC = () => {
const [gitAuthPollingState, setGitAuthPollingState] =
useState<GitAuthPollingState>("idle");

const startPollingGitAuth = useCallback(() => {
setGitAuthPollingState("polling");
}, []);

const { data: gitAuth, error } = useQuery(
versionId
? {
...templateVersionGitAuth(versionId),
refetchInterval: gitAuthPollingState === "polling" ? 1000 : false,
}
: { enabled: false },
);

const allSignedIn = gitAuth?.every((it) => it.authenticated);

useEffect(() => {
if (allSignedIn) {
setGitAuthPollingState("idle");
return;
}

if (gitAuthPollingState !== "polling") {
return;
}
Expand All @@ -74,20 +94,7 @@ const CreateWorkspacePage: FC = () => {
return () => {
clearTimeout(quitPolling);
};
}, [gitAuthPollingState]);

const startPollingGitAuth = useCallback(() => {
setGitAuthPollingState("polling");
}, []);

const { data: gitAuth, error } = useQuery(
versionId
? {
...templateVersionGitAuth(versionId),
refetchInterval: gitAuthPollingState === "polling" ? 1000 : false,
}
: { enabled: false },
);
}, [gitAuthPollingState, allSignedIn]);

return (
<>
Expand Down