Skip to content

Commit def3692

Browse files
authored
Merge pull request #4917 from gitbutlerapp/fix-pr-infinite-loop
fix: Don't update the state with the pre-effect
2 parents 5faa0ac + 3eadc2e commit def3692

File tree

1 file changed

+4
-6
lines changed
  • apps/desktop/src/routes/[projectId]/pull/[number]

1 file changed

+4
-6
lines changed

apps/desktop/src/routes/[projectId]/pull/[number]/+page.svelte

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -12,12 +12,10 @@
1212
import { page } from '$app/stores';
1313
1414
const gitHostListing = getGitHostListingService();
15-
let prs = $state<Readable<PullRequest[]> | undefined>();
16-
let pr = $state<PullRequest>();
17-
$effect.pre(() => {
18-
prs = $gitHostListing?.prs;
19-
pr = $prs?.find((b) => b.number.toString() === $page.params.number);
20-
});
15+
let prs = $derived<Readable<PullRequest[]> | undefined>($gitHostListing?.prs);
16+
let pr = $derived<PullRequest | undefined>(
17+
$prs?.find((b) => b.number.toString() === $page.params.number)
18+
);
2119
</script>
2220

2321
<div class="wrapper">

0 commit comments

Comments
 (0)