Merge lp:~raj-abhilash1/postorius/domainowner into lp:postorius

Proposed by Abhilash Raj
Status: Merged
Merged at revision: 233
Proposed branch: lp:~raj-abhilash1/postorius/domainowner
Merge into: lp:postorius
Diff against target: 47 lines (+2/-13)
3 files modified
src/postorius/forms.py (+0/-3)
src/postorius/templates/postorius/domain_new.html (+0/-9)
src/postorius/views/settings.py (+2/-1)
To merge this branch: bzr merge lp:~raj-abhilash1/postorius/domainowner
Reviewer Review Type Date Requested Status
Terri Approve
Review via email: mp+256242@code.launchpad.net

Description of the change

Add options for serverowner in domain, remove `contact_address` for domains.

To post a comment you must log in.
Revision history for this message
Terri (terriko) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/postorius/forms.py'
--- src/postorius/forms.py 2015-04-14 15:12:54 +0000
+++ src/postorius/forms.py 2015-04-14 22:09:57 +0000
@@ -40,9 +40,6 @@
40 description = forms.CharField(40 description = forms.CharField(
41 label=_('Description'),41 label=_('Description'),
42 required=False)42 required=False)
43 contact_address = forms.EmailField(
44 label=_('Contact Address'),
45 required=False)
4643
47 def clean_mail_host(self):44 def clean_mail_host(self):
48 mail_host = self.cleaned_data['mail_host']45 mail_host = self.cleaned_data['mail_host']
4946
=== modified file 'src/postorius/templates/postorius/domain_new.html'
--- src/postorius/templates/postorius/domain_new.html 2014-10-15 20:59:52 +0000
+++ src/postorius/templates/postorius/domain_new.html 2015-04-14 22:09:57 +0000
@@ -37,15 +37,6 @@
37 </div>37 </div>
38 </div>38 </div>
3939
40 <div class="control-group">
41 {{ form.contact_address.errors }}
42 <label for="id_contact_address" class="control-label">{% trans "Contact Address:" %}</label>
43 <div class="controls">
44 {{ form.contact_address}}
45 <p class="help-block"><strong>{% trans "Example:" %}</strong> {% trans "postmaster@domain.org" %}</p>
46 </div>
47 </div>
48
49 <div class="form-actions">40 <div class="form-actions">
50 <button class="btn btn-success btn-primary" type="submit">{% trans "Create Domain" %}</button>41 <button class="btn btn-success btn-primary" type="submit">{% trans "Create Domain" %}</button>
51 </div>42 </div>
5243
=== modified file 'src/postorius/views/settings.py'
--- src/postorius/views/settings.py 2015-02-09 14:35:44 +0000
+++ src/postorius/views/settings.py 2015-04-14 22:09:57 +0000
@@ -72,7 +72,8 @@
72 if form.is_valid():72 if form.is_valid():
73 domain = Domain(mail_host=form.cleaned_data['mail_host'],73 domain = Domain(mail_host=form.cleaned_data['mail_host'],
74 base_url=form.cleaned_data['web_host'],74 base_url=form.cleaned_data['web_host'],
75 description=form.cleaned_data['description'])75 description=form.cleaned_data['description'],
76 owner=request.user.email)
76 try:77 try:
77 domain.save()78 domain.save()
78 except MailmanApiError:79 except MailmanApiError:

Subscribers

People subscribed via source and target branches