automerge / automerge-repo

MIT License
458 stars 51 forks source link

Un-deprecate the NetworkAdapter class #310

Closed acurrieclark closed 6 months ago

acurrieclark commented 6 months ago

We had previously marked both the StorageAdapter and NetworkAdapter classes as deprecated. However, while the storage adapter offered no functionality, the Network adapter does offer a convenient way to include the EventEmitter functionality which a network adapter requires.

In the future we could likely augment the interface so that it only requires the EventEmitter methods we actually use (emit is the only one, I think), but for now the abstract class is actually useful to us and likely to others too.