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

Proposed by kaputtnik
Status: Merged
Merged at revision: 498
Proposed branch: lp:~widelands-dev/widelands-website/gdpr_1
Merge into: lp:widelands-website
Diff against target: 156 lines (+19/-56)
6 files modified
mainpage/context_processors.py (+0/-6)
mainpage/forms.py (+2/-2)
settings.py (+0/-4)
templates/base.html (+0/-12)
templates/mainpage/legal_notice.html (+17/-11)
templates/right_boxes.html (+0/-21)
To merge this branch: bzr merge lp:~widelands-dev/widelands-website/gdpr_1
Reviewer Review Type Date Requested Status
GunChleoc Approve
Review via email: mp+354355@code.launchpad.net

Commit message

Removed google analytics and PayPal, make some fields optional in the contact form.

Description of the change

After long time absence, i wanted to restart with an easy thing :-)

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

Code LGTM :)

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

I am thinking about giving SirVer some memorial, or other thing which notice him as the generous donor of the online-server. A way to say 'Thank you!' and also a hint for other users on widelands.org.

Maybe a Link the footer, or a special header in the Developers page https://wl.widelands.org/developers/ ?

Revision history for this message
SirVer (sirver) wrote :

> I am thinking about giving SirVer some memorial, or other thing which notice him as the generous donor of the online-server. A way to say 'Thank you!' and also a hint for other users on widelands.org.

Please do not. Giving money is a small contribution compared to giving time and eneregy for the project, I do not want this to be called out.

Revision history for this message
kaputtnik (franku) wrote :

Ok :-)

Will it deploy this evening.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== removed file 'mainpage/context_processors.py'
2--- mainpage/context_processors.py 2018-02-11 14:48:26 +0000
3+++ mainpage/context_processors.py 1970-01-01 00:00:00 +0000
4@@ -1,6 +0,0 @@
5-from django.conf import settings
6-
7-
8-def settings_for_templates(request):
9- context = {'USE_GOOGLE_ANALYTICS': settings.USE_GOOGLE_ANALYTICS,}
10- return context
11
12=== modified file 'mainpage/forms.py'
13--- mainpage/forms.py 2016-12-13 18:28:51 +0000
14+++ mainpage/forms.py 2018-09-05 19:48:01 +0000
15@@ -14,7 +14,7 @@
16
17
18 class ContactForm(forms.Form):
19- surname = forms.CharField(max_length=80)
20- forename = forms.CharField(max_length=80)
21+ surname = forms.CharField(max_length=80, required=False)
22+ forename = forms.CharField(max_length=80, required=False)
23 email = forms.EmailField()
24 inquiry = forms.CharField(widget=forms.Textarea)
25
26=== modified file 'settings.py'
27--- settings.py 2018-05-12 14:01:09 +0000
28+++ settings.py 2018-09-05 19:48:01 +0000
29@@ -142,7 +142,6 @@
30 'django.template.context_processors.static',
31 'django.template.context_processors.tz',
32 'django_messages.context_processors.inbox',
33- 'mainpage.context_processors.settings_for_templates',
34 ],
35 },
36 },
37@@ -275,9 +274,6 @@
38 ########
39 MAPS_PER_PAGE = 10
40
41-
42-USE_GOOGLE_ANALYTICS = False
43-
44 ##############################################
45 ## Recipient(s) who get an email if someone ##
46 ## uses the form on legal notice page ##
47
48=== modified file 'templates/base.html'
49--- templates/base.html 2018-02-11 14:48:26 +0000
50+++ templates/base.html 2018-09-05 19:48:01 +0000
51@@ -25,18 +25,6 @@
52 <link rel="stylesheet" type="text/css" href="{{ MEDIA_URL }}css/base_ielt9.css" />
53 <![endif]-->
54
55- {% if USE_GOOGLE_ANALYTICS %}
56- <!-- Google Analytics -->
57- <script>
58- (function(i,s,o,g,r,a,m){i['GoogleAnalyticsObject']=r;i[r]=i[r]||function(){
59- (i[r].q=i[r].q||[]).push(arguments)},i[r].l=1*new Date();a=s.createElement(o),
60- m=s.getElementsByTagName(o)[0];a.async=1;a.src=g;m.parentNode.insertBefore(a,m)
61- })(window,document,'script','//www.google-analytics.com/analytics.js','ga');
62- ga('create', 'UA-42120570-2', 'widelands.org');
63- ga('send', 'pageview');
64- </script>
65- {% endif %}
66-
67 <!-- Javascript Bread & Butter Scripts -->
68 <script type="text/javascript" src="{{ MEDIA_URL }}js/jquery-3.2.1.js"></script>
69 <script type="text/javascript" src="{{ MEDIA_URL }}js/jquery-ui.min.js"></script>
70
71=== modified file 'templates/mainpage/legal_notice.html'
72--- templates/mainpage/legal_notice.html 2016-02-29 23:14:56 +0000
73+++ templates/mainpage/legal_notice.html 2018-09-05 19:48:01 +0000
74@@ -12,7 +12,8 @@
75
76 <h2>Project lead</h2>
77 <ul>
78- <li>Holger Rapp</li>
79+ <li>GunChleoc (Since 2016)</li>
80+ <li>Holger Rapp (SirVer) (2001 – 2016)</li>
81 </ul>
82
83 <h2>Contact</h2>
84@@ -28,25 +29,30 @@
85 </ul>
86
87 <form action="/legal_notice/" method="post">{% csrf_token %}
88- {% if form.errors %}
89- <p class="errormessage" style="text-align: left;">Please fill out all fields!</p>
90- {% endif %}
91 <table>
92 <tr>
93- <td><label for="id_forename">First name: </label></td>
94- <td><input id="id_forename" type="text" name="forename" maxlength="80" required></td>
95+ <td><label for="id_forename">First name (optional): </label></td>
96+ <td><input id="id_forename" type="text" name="forename" maxlength="80"></td>
97 </tr>
98 <tr>
99- <td><label for="id_surname">Last name: </label></td>
100- <td><input id="id_surname" type="text" name="surname" maxlength="80" required></td>
101+ <td><label for="id_surname">Last name (optional): </label></td>
102+ <td><input id="id_surname" type="text" name="surname" maxlength="80"></td>
103 </tr>
104 <tr>
105 <td><label for="id_email">Email:</label></td>
106- <td><input type="text" name="email" id="id_email" required></td>
107+ <td><input type="email" name="email" id="id_email" required>
108+ {% for error in form.email.errors %}
109+ <span class="errormessage">{{ error }}</span>
110+ {% endfor %}
111+ </td>
112 </tr>
113 <tr>
114- <td><label for="id_inquiry">Inquiry: </label></td>
115- <td><textarea id="id_inquiry" rows="10" cols="40" name="inquiry" required></textarea></td>
116+ <td><label for="id_inquiry">Inquiry: </label>{{ form.inquiry.errors }}</td>
117+ <td><textarea id="id_inquiry" rows="10" cols="40" name="inquiry" required></textarea>
118+ {% for error in form.inquiry.errors %}
119+ <span class="errormessage">{{ error }}</span>
120+ {% endfor %}
121+ </td>
122 </tr>
123 <tr>
124 <td></td>
125
126=== modified file 'templates/right_boxes.html'
127--- templates/right_boxes.html 2018-04-13 07:31:42 +0000
128+++ templates/right_boxes.html 2018-09-05 19:48:01 +0000
129@@ -11,27 +11,6 @@
130 {% load wlprofile_extras wlpoll_extras wlevents_extras %}
131 {% load pybb_extras %}
132
133-<!-- Donation Box -->
134-<div class="columnModule">
135- <h3>Donation</h3>
136- <div class="columnModuleBox center">
137- <p>
138- Help us to pay our server!
139- <br />
140- (: Consider a donation :)
141- </p>
142- <form action="https://www.paypal.com/cgi-bin/webscr" method="post">
143- <input type="hidden" name="cmd" value="_s-xclick" />
144- <input type="hidden" name="lc" value="GB" />
145- <input type="hidden" name="hosted_button_id" value="JH5R7YHSVRMRG" />
146- <input type="image" class="donate" src="https://www.paypal.com/en_US/i/btn/btn_donate_SM.gif" name="submit" alt="PayPal - The safer, easier way to pay online!" />
147- <img alt="" src="https://www.paypal.com/de_DE/i/scr/pixel.gif" width="1" height="1" />
148- </form>
149- <br /><br />
150- </div>
151-</div>
152-
153-
154 <!-- Social Media Box -->
155 <div class="columnModule">
156 <h3>Social Media</h3>

Subscribers

People subscribed via source and target branches