Skip to content

Conversation

@arcoraven
Copy link
Contributor

@arcoraven arcoraven commented Apr 24, 2025

Removes two unneeded listeners. These caches are already cleared by the endpoints that update the data models.


PR-Codex overview

This PR primarily focuses on removing unused listeners and simplifying cache clearing functionality across various components of the application.

Detailed summary

  • Deleted config-listener.ts, webhook-listener.ts, and update-tx-listener.ts.
  • Simplified clearCache function by removing the _service parameter.
  • Updated clearCacheCron to remove the _service parameter and adjusted its usage in the cron job.
  • Removed calls to the deleted listeners in worker/index.ts and server/index.ts.

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

@zeet-co
Copy link

zeet-co bot commented Apr 24, 2025

We're building your pull request over on Zeet.
Click me for more info about your build and deployment.
Once built, this branch can be tested at: https://tw-unreal-demo-engine-vwn0-ph-rem-ecb512.engine-aws-usw2.zeet.app before merging 😉

import { keypairCache } from "./keypair";

export const clearCache = async (
_service: (typeof env)["LOG_SERVICES"][0],
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unused var

@arcoraven arcoraven merged commit 377656d into main Apr 24, 2025
7 checks passed
@arcoraven arcoraven deleted the ph/remove-unneeded-listeners branch April 24, 2025 16:10
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.

4 participants