{"action":"opened","issue":{"url":"https://api.github.com/repos/phetsims/rosetta/issues/80","labels_url":"https://api.github.com/repos/phetsims/rosetta/issues/80/labels{/name}","comments_url":"https://api.github.com/repos/phetsims/rosetta/issues/80/comments","events_url":"https://api.github.com/repos/phetsims/rosetta/issues/80/events","html_url":"https://github.com/phetsims/rosetta/issues/80","id":108721138,"number":80,"title":"Should session be checked on every page serve?","user":{"login":"jbphet","id":3082831,"avatar_url":"https://avatars.githubusercontent.com/u/3082831?v=3","gravatar_id":"","url":"https://api.github.com/users/jbphet","html_url":"https://github.com/jbphet","followers_url":"https://api.github.com/users/jbphet/followers","following_url":"https://api.github.com/users/jbphet/following{/other_user}","gists_url":"https://api.github.com/users/jbphet/gists{/gist_id}","starred_url":"https://api.github.com/users/jbphet/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jbphet/subscriptions","organizations_url":"https://api.github.com/users/jbphet/orgs","repos_url":"https://api.github.com/users/jbphet/repos","events_url":"https://api.github.com/users/jbphet/events{/privacy}","received_events_url":"https://api.github.com/users/jbphet/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-09-28T18:41:54Z","updated_at":"2015-09-28T18:41:54Z","closed_at":null,"body":"There is a comment in routes.js that says:\r\n\r\n```js\r\n/*\r\n .\r\n .\r\n .\r\n * TODO: As of mid-Feb 2015, this checks the session on every page serve.  It would probably be better to do some local\r\n * management of sessions, and only re-check them periodically.\r\n .\r\n .\r\n .\r\n */\r\n```\r\n\r\nI believe I wrote this comment originally, and did so because I was concerned that checking session status for every page might make the translation utility operate a little slowly.  @aaronsamuel137 probably has a better feel for whether this is true, since he's been doing the bulk of the development and testing on this.  We should either remove this comment, add some local session management, or create a new issue about adding local session management and mark it with the \"not for 1.0\" label.  I'll leave it to @aaronsamuel137's discretion as to which option makes the most sense."}}