[opensource-dev] Review Request: Change pre-login viewer display URL to prepare for new community information display

[Thickbrick Sleaford]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/399/#review883
-----------------------------------------------------------



indra/newview/app_settings/settings.xml
<http://codereview.secondlife.com/r/399/#comment922>

    Setting the default value to something other than an empty string would override the selected grid's login page in LLGridManager::getGridInfo and LLGridManager::getLoginPage, which is probably not the desired result. In fact, I wonder why this setting is persistent at all. This should probably be renamed to CmdLineLoginPage, to be consistent with CmdLineLoginURI etc.


- Thickbrick


On July 16, 2011, 6:12 a.m., Oz Linden wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/399/
> -----------------------------------------------------------
> 
> (Updated July 16, 2011, 6:12 a.m.)
> 
> 
> Review request for Viewer.
> 
> 
> Summary
> -------
> 
> This changes the URL from which the upper part of the viewer window is loaded prior to login.
> 
> At the moment, the new URL is internally redirected to the same content as before, but shortly it will have new community and activity information on what's going on in Second Life.
> 
> 
> This addresses bug storm-1510.
>     http://jira.secondlife.com/browse/storm-1510
> 
> 
> Diffs
> -----
> 
>   indra/newview/app_settings/settings.xml c6f2f4af65e5 
>   indra/newview/llviewernetwork.cpp c6f2f4af65e5 
>   indra/newview/tests/llviewernetwork_test.cpp c6f2f4af65e5 
> 
> Diff: http://codereview.secondlife.com/r/399/diff
> 
> 
> Testing
> -------
> 
> Confirmed using the log file that the new URL is being requested.
> Changed the settings file locally to point to an internal prototype of the new screen and confirmed that it is displayed correctly.
> 
> 
> Thanks,
> 
> Oz
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110716/0880555e/attachment.htm