Skip to content

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}

@socket-security
Copy link

@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.

2 participants