::destroy dispatches ::destroy?

[iku000888]

Hi @oliyh !

This has been really useful, thanks!

I noticed that calling re-graph.core/destroy throws a runtime error, and seemed like commenting this line out made the error stop.

re-graph/src/re_graph/core.cljs

Line 165 in 2b8f8fd

:dispatch [::destroy instance-name]}

I have not looked into the internals much, but it would be awesome if you could highlight some intentions.

More than happy to provide more details as well.

[oliyh]

Hi,
The idea is to unsubscribe from any existing subscriptions, and to allow that to happen (as the unsubscribe will require the websocket / instance) before actually destroying the instance.

Thank you for the pull request, it looks good!