Merge lp:~widelands-dev/widelands-website/post_registration_cleanup into lp:widelands-website

Proposed by kaputtnik
Status: Merged
Merged at revision: 526
Proposed branch: lp:~widelands-dev/widelands-website/post_registration_cleanup
Merge into: lp:widelands-website
Diff against target: 51 lines (+4/-22)
3 files modified
templates/django_registration/activate.html (+0/-21)
templates/django_registration/activation_failed.html (+3/-0)
urls.py (+1/-1)
To merge this branch: bzr merge lp:~widelands-dev/widelands-website/post_registration_cleanup
Reviewer Review Type Date Requested Status
GunChleoc Approve
Review via email: mp+364274@code.launchpad.net

Description of the change

The latest server errors occur because:

1. I have overlooked renaming a template, so it was missing
2. Added the missing template as a new template... this was accidentally done this morning on alpha instead of wl.widelands.org. Took me two hours to find this mistake :D

So this branch is just a small cleanup.

- removed a (now) unused template
- fixed a url

To be clear i have also checked if registering is working, and the result is fine.

To post a comment you must log in.
Revision history for this message
GunChleoc (gunchleoc) wrote :

Ouch, sounds like you did not have fun.

Code looks good :)

review: Approve
Revision history for this message
kaputtnik (franku) wrote :

That's always a bit thrilling...

Merged and deployed.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== removed file 'templates/django_registration/activate.html'
2--- templates/django_registration/activate.html 2019-03-06 16:37:05 +0000
3+++ templates/django_registration/activate.html 1970-01-01 00:00:00 +0000
4@@ -1,21 +0,0 @@
5-{% extends "mainpage/base.html" %}
6-{% comment %}
7-vim:ft=htmldjango:
8-{% endcomment %}
9-
10-{% block title %}
11-Activation - {{ block.super }}
12-{% endblock %}
13-
14-{% block content_header %}
15- <h1>Activation</h1>
16-{% endblock %}
17-
18-{% block content_main %}
19-<div class="blogEntry">
20- <p class="errormessage">
21- An error occured: Either this account has already been activated or activation key is invalid.<br />
22- Try <a href="{% url 'login' %}">logging in</a>.
23- </p>
24-</div>
25-{% endblock %}
26
27=== modified file 'templates/django_registration/activation_failed.html'
28--- templates/django_registration/activation_failed.html 2019-03-11 06:23:17 +0000
29+++ templates/django_registration/activation_failed.html 2019-03-11 19:58:28 +0000
30@@ -13,5 +13,8 @@
31 <p>
32 An error occured during activation: {{ activation_error.message }}
33 </p>
34+ <p>
35+ You can try to <a href="{% url 'login' %}">log in</a>.
36+ </p>
37 </div>
38 {% endblock %}
39
40=== modified file 'urls.py'
41--- urls.py 2019-03-03 15:17:58 +0000
42+++ urls.py 2019-03-11 19:58:28 +0000
43@@ -25,7 +25,7 @@
44 url(r'^accounts/register/$', RegistrationView.as_view(
45 form_class=RegistrationWithCaptchaForm), name='django_registration_register'),
46 url(r'^accounts/', include('django_registration.backends.activation.urls')),
47- url('^', include('django.contrib.auth.urls')),
48+ url(r'^', include('django.contrib.auth.urls')),
49
50 url(r'^ratings/', include('star_ratings.urls', namespace='ratings', app_name='ratings')),
51 # Formerly 3rd party

Subscribers

People subscribed via source and target branches