Skip to content
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.

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

fix(experimental): Invalidate inflight cache in thirdweb rpcClient #823

Closed
wants to merge 2 commits into from

Conversation

arcoraven
Copy link
Contributor

@arcoraven arcoraven commented Jan 1, 2025

PR-Codex overview

This PR focuses on updating dependencies and enhancing the postinstall script in the package.json, along with modifications in the rpc.ts file to improve request handling.

Detailed summary

  • Added postinstall script to package.json for patch-package.
  • Updated dependencies in package.json, including:
    • Added patch-package and postinstall-postinstall.
  • Modified rpc.ts to clear inflightRequests on completion of requests.
  • Updated several packages in yarn.lock.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@arcoraven arcoraven added the do not merge Do not merge the branch label Jan 1, 2025
@arcoraven
Copy link
Contributor Author

THIS BRANCH IS EXPERIMENTAL. DO NOT MERGE.

If this fix works, the correct fix is to push it up to the thirdweb sdk.

@arcoraven arcoraven closed this Jan 4, 2025
@arcoraven arcoraven deleted the ph/clearInflightCache__patch branch January 4, 2025 02:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge Do not merge the branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant