digidem / mapeo-core-next

The upcoming version of Mapeo Core
MIT License
7 stars 1 forks source link

chore: simplify `SyncApi` "is syncing" logic #610

Closed EvanHahn closed 2 months ago

EvanHahn commented 2 months ago

This change should have no impact.

SyncApi needs to know whether sync is enabled. Currently, it does this in a roundabout way: appending a value to a set and checking for that value.

This change replaces that with a simpler boolean flag.

I think this is a useful change on its own, but it'll be useful as part of #576 too.