smart-facility / cognicity-rem-web

Cognicity REM web site
Other
0 stars 0 forks source link

login page doesn't set site icon to be the new one #32

Closed matthewberryman closed 8 years ago

matthewberryman commented 8 years ago

https://github.com/smart-facility/cognicity-rem-web/issues/31 After logging in the new one is displayed correctly. cc @talltom

matthewberryman commented 8 years ago

Dang, added this to the wrong project.

matthewberryman commented 8 years ago

@benatwork99 I thought I had fixed with https://github.com/smart-facility/cognicity-rem-server/commit/98383196545efdf53074a10adbd0d64065cac37d#diff-b04fa3f30541b923db131e9aed6eb945 but that code ends up actually doing a redirect—I had missed this in testing on rem-dev because I had entered the main url again, but it is consistent on logon, and now with your new code it's easier to test as you can logout much more easily ;)

I guess the redirect is happening because of an interplay with the node.js server code and in particular its redirect code. Can you please take a look? Note I reverted with https://github.com/smart-facility/cognicity-rem-web/commit/5351d0034ae81551896e00a9eedd341cd9627ec4