Closed tomjn closed 9 months ago
Note that I tried updating Angular further but 1.2 moves the router into its own package making an easy upgrade a bit more involved
1.2 required a routing JS be included, moving to 1.3 will require refactoring the main controller which is a bit more involved, leaving it as out of scope
Note there's a subtle rotation of the card when hovering now:
Previously it stored the socket ID on a connection, but if you Open a new tab there's multiple connections with the same ID, so when it cleaned up one socket the other was orphaned.
Now there's an array of socket IDs!
I'd love to rebuild the UI in React with a store, just needs time which is easier said than done
This should resolve #35 #36 and lay some groundwork for #21
It also upgrade socketIO, Angular, and Node