Do not block when port forwarding is not enabled

This commit is contained in:
Arnaud 2024-11-11 09:47:09 +07:00
parent 20f5b75a9f
commit a577db89fa
No known key found for this signature in database
GPG Key ID: 69D6CE281FCAE663

View File

@ -37,14 +37,10 @@ export function HealthChecks({ online, onStepValid }: Props) {
useEffect(
() => {
if (codex.isSuccess) {
persistence.refetch();
portForwarding.refetch().then(({ data }) => {
onStepValid(data?.reachable || false);
});
} else {
onStepValid(false);
}
persistence.refetch();
portForwarding.refetch();
onStepValid(codex.isSuccess);
},
// We really do not want to add persistence and portForwarding as
// dependencies because it will cause a re-render every time.
@ -163,18 +159,6 @@ export function HealthChecks({ online, onStepValid }: Props) {
</span>
Internet connection
</li>
<li>
<span>
{portForwarding.isFetching ? (
<Spinner></Spinner>
) : portForwarding.enabled ? (
<SuccessCircleIcon></SuccessCircleIcon>
) : (
<ErrorCircleIcon width={16} />
)}
</span>
Port forwarding
</li>
<li>
<span>
{codex.isFetching ? (
@ -187,6 +171,18 @@ export function HealthChecks({ online, onStepValid }: Props) {
</span>
Codex connection
</li>
<li>
<span>
{portForwarding.isFetching ? (
<Spinner></Spinner>
) : portForwarding.enabled ? (
<SuccessCircleIcon></SuccessCircleIcon>
) : (
<WarningIcon width={16} />
)}
</span>
Port forwarding
</li>
<li>
<span>
{persistence.isFetching ? (