Skip to content

fix: noop in example#512

Closed
jeffbrownn wants to merge 1 commit intoProjectOpenSea:mainfrom
jeffbrownn:main
Closed

fix: noop in example#512
jeffbrownn wants to merge 1 commit intoProjectOpenSea:mainfrom
jeffbrownn:main

Conversation

@jeffbrownn
Copy link
Copy Markdown

Change Overview

Fixes an issue in the README where the noop callback is used without being defined or imported in the unsubscribe example.
The previous example would not compile when copied directly.
It has been replaced with an inline empty function to keep the example simple, valid, and copy-paste friendly.

@github-actions
Copy link
Copy Markdown

This PR has been automatically marked as stale because it has not had recent activity. It will be closed in 14 days if no further activity occurs. Thank you for your contributions. If you believe this was a mistake, please comment.

@github-actions github-actions Bot added the Stale label Mar 28, 2026
@ryanio
Copy link
Copy Markdown
Collaborator

ryanio commented Apr 7, 2026

Thanks for this contribution! We've recreated this fix in our internal monorepo and it will be included in the next release sync. Appreciate you flagging it!

@ryanio ryanio closed this Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants