Redirect and show message after editing profile

[Jef van Schendel]

After you edit your profile, you are redirected to the root page.

What should happen: the site goes to your profile page and shows a notification on top using our shiny new messaging system.

[Caleb Davis]

ooh, ooh, lemme do it.

[Caleb Davis]

ok, shiny happy successful edit profile message appears without extra navigation. This was a change of route name and an add_message(...)

[https://gitorious.org/_{cfdv/mediagoblin/cfdvs-mediagoblin/commit/d9ed098e74617916de85142400b00af973a49b76](https://gitorious.org/}cfdv/mediagoblin/cfdvs-mediagoblin/commit/d9ed098e74617916de85142400b00af973a49b76)

[Christopher Allan Webber]

Merged!

[Elrond]

Reopening after talking to Celeb:

Jef suggested to redirect to the user's homepage. The current code redirects to the "edit profile" page. ;)

Should be easy to fix.

[Caleb Davis]

I changed the redirect after form validation to point to the user home page instead of the edit profile page.

when you merge, you'll probably want to

• user=user['username'])
• user=edit_username)

[https://gitorious.org/_{cfdv/mediagoblin/cfdvs-mediagoblin/commits/b404_v2_redirect_after_profile_edit](https://gitorious.org/}cfdv/mediagoblin/cfdvs-mediagoblin/commits/b404_v2_redirect_after_profile_edit)

[Christopher Allan Webber]

404 merge not found, I mean merged.

[Caleb Davis]

I get an error upon submitting the profile edit

  File "/home/vdfc/src/mediagoblin/mediagoblin/edit/views.py", line 111, in edit_profile
    username=edit_username)
  File "/home/vdfc/src/mediagoblin/mediagoblin/util.py", line 136, in redirect
    return exc.HTTPFound(location=request.urlgen(*args, **kwargs))
  File "/home/vdfc/src/mediagoblin/eggs/Routes-1.12.3-py2.6.egg/routes/util.py", line 420, in __call__
    (args, kargs))
GenerationException: Could not generate URL. Called with args: ('mediagoblin.user_pages.user_home',) {'username': u'dev_user2'}

I'm not sure how to re-open the bug. Let's do this

diff --git a/mediagoblin/edit/views.py b/mediagoblin/edit/views.py
index a3326b2..3bcf788 100644
--- a/mediagoblin/edit/views.py
+++ b/mediagoblin/edit/views.py
@@ -108,7 +108,7 @@ def edit_profile(request):
                                 'Profile edited!')
             return redirect(request,
                            'mediagoblin.user_pages.user_home',
-                           username=edit_username)
+                           user=edit_username)

     return render_to_response(
         request,

or, [https://gitorious.org/_{cfdv/mediagoblin/cfdvs-mediagoblin/commits/b404_v3_edit_profile_submit_fail](https://gitorious.org/}cfdv/mediagoblin/cfdvs-mediagoblin/commits/b404_v3_edit_profile_submit_fail)

[Christopher Allan Webber]

Added you as manager, you should be able to reopen bugs now.

Anyway, good call, and merged! Thanks!

[Will Kahn-Greene]

The original url for this bug was ​http://bugs.foocorp.net/issues/404 .