Skip to content

Fix bugs - #280

Merged
tdresser merged 4 commits into
mainfrom
fix_bugs_again
Dec 11, 2025
Merged

tdresser merged 4 commits into
mainfrom
fix_bugs_again

Conversation

@tdresser

Copy link
Copy Markdown
Collaborator

No description provided.

startWith(0),
shareReplay({ bufferSize: 1, refCount: true }),
);
// TODO: does forcing an early subscribe get this to happen earlier, avoiding an

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I have a better solution than this, but haven't had time to test it yet.

This does do the job!

@tdresser
tdresser requested a review from IanEdington December 11, 2025 01:21

@IanEdington IanEdington left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@tdresser
tdresser merged commit 7a1df95 into main Dec 11, 2025
3 checks passed
@tdresser
tdresser deleted the fix_bugs_again branch December 11, 2025 05:43
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.

2 participants