Moderator functionality not complete

[ccotter]

As far as I can tell, some features promised by the SessionModerator interface are not actually provided by the coweb Java server. For example, the SessionModerator.onServiceResponse callback is never actually called by the server anywhere.

This issue shall list missing moderator features, and track their eventual implementation.

Missing/Need more information

• onServiceResponse not actually called anywhere.
• onSync - supposedly (from reading existing docs) the return value dictates whether or not the sync event should be forwarded to bots, although the return value of onSync is discarded by the server. Not sure if the docs are wrong or not, though.

[vinomaster]

Fix OnSync to no longer communicate to bots.

We then need to update CoMap to use Moderator.

[ccotter]

Just for reference (mainly my own):

CoMap subscribes to a service /bot/zipvisits. The server side bot (Java or python) listens for sync events (not service messages) sent by clients. Every five seconds, the bot sends a service message to clients with updated visit counts.

From now on, bots will no longer be able to know about sync events (mentinoed by @vinomaster). The moderator will do exactly what the comap bot does, except through sync channels and not service channels.

[ccotter]

Implementation is pretty much complete, but I need to document the API and how the CoMap demo works (probably with a tutorial of sorts).

[ccotter]

There is a lot of overlap in the javadocs for moderator and the sphinx docs. I'm going to move all SessionModerator documentation in the javadoc and point users to the javadocs page in the sphinx doc page for the moderator. In addition, however, I'l describe how to use the moderator in the sphinx page.

[ccotter]

All features have been added, so this issue is resolved.