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

[nip66] maintenance on nip66/adapters #714

Open
13 tasks
dskvr opened this issue Dec 17, 2024 · 0 comments
Open
13 tasks

[nip66] maintenance on nip66/adapters #714

dskvr opened this issue Dec 17, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@dskvr
Copy link
Collaborator

dskvr commented Dec 17, 2024

  • remove non-functional adapters
  • Eliminate deprecated Workers class dependency (only one method is being used since change lifecycle management, the entire class can be deleted once the dependency on that method can be removed)
  • Eliminate callback hell between service <-> adapter that was inadvertently ushered into existence while identifying best caching strategy.
  • eliminate head-spin between Adapter <-> ${Type}Adapter child classes.
  • Clarify CacheAdapter interface
  • Clarify WebsocketAdapter interface
  • DRY up Monitor service (wet af)
  • Update existing tests
  • Write tests for all core classes.
  • Write tests for all utils
  • Write tests for NostrTools and NostrSqlite (worker-relay) adapters.

discoveries

  • explore converting subscritbe and fetch in Services base class to a generator.
  • explore extending that generator from gui to yield/inject events store
@dskvr dskvr added the bug Something isn't working label Dec 17, 2024
@dskvr dskvr changed the title [nip66] maintenance on library/adapters [nip66] maintenance on nip66/adapters Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant