launchpad-reviewers team mailing list archive

[Raphaël Badin <raphael.badin@xxxxxxxxxxxxx>]

The proposal to merge lp:~rvb/maas/maas-longpoll-js into lp:maas has been updated.

Commit Message changed to:

This branch adds the required view/utility to deal with longpoll on the server side.  It adds:
- the required configuration options
- a simple view which will act as a proxy to a txlongpoll server.  Note that this will be used in dev mode only.  In production, apache will do that.  Also, in the future, we might want, in dev mode, to setup a tiny and lightweight http server that will act as a frontend to django's dev server and txlongpoll.
- view.get_longpoll_context: a simply utility used to populate a context with the information required to connect longpoll on the js side.

Note that LONGPOLL is disabled on the dev instance (LONGPOLL_URL = None in demo.py) by default because having it on breaks Django's auto reload feature.

For more details, see:
https://code.launchpad.net/~rvb/maas/maas-longpoll-js/+merge/97204
-- 
https://code.launchpad.net/~rvb/maas/maas-longpoll-js/+merge/97204
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~rvb/maas/maas-longpoll-js into lp:maas.