Circular dependencies

[mktcode]

I want to use my local copy of snapshot.js when working on the hub and/or UI. I use yarn link for that but I get this error:

Re-building it or running dev mode doesn't work. But what I noticed is this:

Circular dependencies... Well, that doesn't sound good. Some more searching later:

I checked out some older commits and it is like that for more than a year... at least. Imho already far too long and we just build up more and more technical debt. This should be fixed asap.

[samuveth]

Interesting. Not sure if this helps as a quick fix in your case but for the Ui I use yalc to add my local version of snapshot.js

[mktcode]

Not such a big fan of yalc (after a quick read) since it creates/touches files (yalc.lock/package.json). I prefer the yarn/npm native solution. The problem here needs to be fixed anyway. It's enough for now to just fix the one circular dependency regarding voting. Will submit a PR in a bit. Technically this would introduce a breaking change -> major version update. Score repo would need this update then: