edx/edx-platform — pull request #582 by adampalay (2013-08-05 19:18:20+00:00)
Title: Fix/adam/redirect

@cdodge @feanil

I had pinpointed the wrong file to change the redirects in. Here are the lms views that redirect to "/". I'm changing them to redirect to "/dashboard." The CMS view should be changed back to redirect to "/". (or we can rebase that earlier commit out). "/dashboard" is not a valid url for cms anyway.

It turns out that the original "edge" view wasn't even pointing to a valid template. It returned "render_to_response('university_profiles/edge.html', {})". But "edge.html" was in a directory called "university_profile", no "s". So that would have returned a 404 if it were ever called.

When I test the the "/courses" url out on stage, it redirects to "/" and then falls into that loop. Originally I had thought to redirect it to the dashboard view by calling "redirect(reverse('dashboard'))". But, as we're not using branding, it made more sense just directly to call "student.views.index" rather than have an extra bit of logic.

@chrisndodge and @brianhw ?