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

Richard Nelson richard at lindenlab.com
Mon Jul 18 10:02:42 PDT 2011


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

Ship it!


Looks good, let's try it without the change to settings.xml, though.

- Richard


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/20110718/aac9c86b/attachment.htm 


More information about the opensource-dev mailing list