{"action":"created","issue":{"url":"https://api.github.com/repos/choonkending/react-webpack-node/issues/62","labels_url":"https://api.github.com/repos/choonkending/react-webpack-node/issues/62/labels{/name}","comments_url":"https://api.github.com/repos/choonkending/react-webpack-node/issues/62/comments","events_url":"https://api.github.com/repos/choonkending/react-webpack-node/issues/62/events","html_url":"https://github.com/choonkending/react-webpack-node/pull/62","id":102518384,"number":62,"title":"Separate layouts & bug showing req.user object","user":{"login":"joshhornby","id":5455767,"avatar_url":"https://avatars.githubusercontent.com/u/5455767?v=3","gravatar_id":"","url":"https://api.github.com/users/joshhornby","html_url":"https://github.com/joshhornby","followers_url":"https://api.github.com/users/joshhornby/followers","following_url":"https://api.github.com/users/joshhornby/following{/other_user}","gists_url":"https://api.github.com/users/joshhornby/gists{/gist_id}","starred_url":"https://api.github.com/users/joshhornby/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joshhornby/subscriptions","organizations_url":"https://api.github.com/users/joshhornby/orgs","repos_url":"https://api.github.com/users/joshhornby/repos","events_url":"https://api.github.com/users/joshhornby/events{/privacy}","received_events_url":"https://api.github.com/users/joshhornby/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-08-22T08:57:08Z","updated_at":"2015-11-17T20:45:19Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/choonkending/react-webpack-node/pulls/62","html_url":"https://github.com/choonkending/react-webpack-node/pull/62","diff_url":"https://github.com/choonkending/react-webpack-node/pull/62.diff","patch_url":"https://github.com/choonkending/react-webpack-node/pull/62.patch"},"body":"# Related Issues\r\n\r\n#56 #57 #58 \r\n\r\nThis PR shows two things\r\n\r\n1) The ability to split the login route into a different 'layout' - this will allow theming etc.\r\n2) An example showing the issues stated in issue #57 - Once you login you will be auto redirected to the dashboard (https://github.com/choonkending/react-webpack-node/compare/master...joshhornby:feature/layout?expand=1#diff-129f1ad504e80d839c390c823d8ad3c2R70) You will notice you should see the users email in the nav bar (https://github.com/choonkending/react-webpack-node/compare/master...joshhornby:feature/layout?expand=1#diff-129f1ad504e80d839c390c823d8ad3c2R70) - this has been added to the `user` object in the server routes file (https://github.com/choonkending/react-webpack-node/compare/master...joshhornby:feature/layout?expand=1#diff-99de27cb2ded592bf6ae68b3d132eaceR62) But the email is only shown AFTER a refresh of the page - this is because after the redirect the user state only contains the following \r\n\r\n`Object {authenticated: true, isWaiting: false}`\r\n\r\nWhich is set here https://github.com/choonkending/react-webpack-node/compare/master...joshhornby:feature/layout?expand=1#diff-129f1ad504e80d839c390c823d8ad3c2R67\r\n\r\n# Todos\r\n\r\n- [ ] I don't love the way the router is passed into the store - it just doesn't feel correct\r\n- [ ] Make the redirect dynamic as seen in this following example - https://github.com/rackt/react-router/blob/master/examples/auth-flow/app.js#L79\r\n\r\n@choonkending Looking forward to hearing your feedback, especially regarding point 2. This may also be related to issue #58 as that should handle the redirect for us come to think of it, but as outlined on there it doesn't always seem to be fired."},"comment":{"url":"https://api.github.com/repos/choonkending/react-webpack-node/issues/comments/157500953","html_url":"https://github.com/choonkending/react-webpack-node/pull/62#issuecomment-157500953","issue_url":"https://api.github.com/repos/choonkending/react-webpack-node/issues/62","id":157500953,"user":{"login":"choonkending","id":3787885,"avatar_url":"https://avatars.githubusercontent.com/u/3787885?v=3","gravatar_id":"","url":"https://api.github.com/users/choonkending","html_url":"https://github.com/choonkending","followers_url":"https://api.github.com/users/choonkending/followers","following_url":"https://api.github.com/users/choonkending/following{/other_user}","gists_url":"https://api.github.com/users/choonkending/gists{/gist_id}","starred_url":"https://api.github.com/users/choonkending/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/choonkending/subscriptions","organizations_url":"https://api.github.com/users/choonkending/orgs","repos_url":"https://api.github.com/users/choonkending/repos","events_url":"https://api.github.com/users/choonkending/events{/privacy}","received_events_url":"https://api.github.com/users/choonkending/received_events","type":"User","site_admin":false},"created_at":"2015-11-17T20:45:19Z","updated_at":"2015-11-17T20:45:19Z","body":"@joshhornby I think the repository has changed significantly since it's deploy. We might need to update this branch (but hold off on this, I still wish to restructure the redux folder into the main folder and remove unneeded dependencies) :)"}}