Postorius

Merge lp:~pskoli93/postorius/RedirectFix into lp:postorius

  1. RedirectFix
  2. Merge into trunk
Proposed by Pavan Koli
Status: Needs review
Proposed branch: lp:~pskoli93/postorius/RedirectFix
Merge into: lp:postorius
Diff against target: 12 lines (+1/-1)
1 file modified
src/postorius/views/list.py (+1/-1)
To merge this branch: bzr merge lp:~pskoli93/postorius/RedirectFix
Reviewer Review Type Date Requested Status
Florian Fuchs Approve
Review via email: mp+256424@code.launchpad.net

Description of the change

On choosing any list, when the list owner wants to unsubscribe from a list(where he had subscribed previously). On clicking the unsubscribe the owner is redirected to list_members page, instead of the list_summary page.

To post a comment you must log in.
Revision history for this message
Florian Fuchs (flo-fuchs) wrote :

Hi Pavan,

thanks for that fix! Only I just readlized I had already fixed that in another branch.

Marking it as approved...

Florian

review: Approve
Reply

Unmerged revisions

242. By pkoli <email address hidden>

Redirect fix

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/postorius/views/list.py'
2--- src/postorius/views/list.py 2015-04-15 23:44:05 +0000
3+++ src/postorius/views/list.py 2015-04-16 06:40:49 +0000
4@@ -228,7 +228,7 @@
5 return utils.render_api_error(request)
6 except ValueError, e:
7 messages.error(request, e)
8- return redirect('list_members', self.mailing_list.list_id)
9+ return redirect('list_summary', self.mailing_list.list_id)
10
11
12 class ListMassSubscribeView(MailingListView):

Subscribers

People subscribed via source and target branches