Skip to content

Commit 85b09a1

Browse files
authored
Merge pull request #25 from tbrlpld/django-rest-user-field
Hide user field in api front-end and set default
2 parents 9628a78 + d9ee51b commit 85b09a1

File tree

4 files changed

+30
-2
lines changed

4 files changed

+30
-2
lines changed

days/069-072-django-rest/demo/api/serializers.py

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,13 @@
55

66
class QuoteSerializer(serializers.ModelSerializer):
77

8+
# This is not included in the videos. Without this setting, it was possible
9+
# to set the user value to something other than the currently logged-in
10+
# user. This setting hides the user field from the form in the API frontend
11+
# and sets the currently logged-in users as the field value by default.
12+
# See also: https://stackoverflow.com/a/53193276
13+
user = serializers.HiddenField(default=serializers.CurrentUserDefault())
14+
815
class Meta:
916
model = Quote
1017
fields = ('quote', 'author', 'source', 'cover', 'user')

days/069-072-django-rest/demo/mysite/templates/django_registration/registration_form.html

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,19 +14,40 @@ <h2>Sign up today!</h2>
1414
<div class="mui-textfield">
1515
{{ form.username.label }}
1616
{{ form.username }}
17+
<div class="mui--text-danger">
18+
{% for error in form.username.errors %}
19+
{{ error }}
20+
{% endfor %}
21+
</div>
1722
</div>
1823
<div class="mui-textfield">
1924
{{ form.email.label }}
2025
{{ form.email }}
26+
<div class="mui--text-danger">
27+
{% for error in form.email.errors %}
28+
{{ error }}
29+
{% endfor %}
30+
</div>
2131
</div>
2232
<div class="mui-textfield">
2333
{{ form.password1.label }}
2434
{{ form.password1 }}
35+
<div class="mui--text-danger">
36+
{% for error in form.password1.errors %}
37+
{{ error }}
38+
{% endfor %}
39+
</div>
2540
</div>
2641
<div class="mui-textfield">
2742
{{ form.password2.label }}
2843
{{ form.password2 }}
44+
<div class="mui--text-danger">
45+
{% for error in form.password2.errors %}
46+
{{ error }}
47+
{% endfor %}
48+
</div>
2949
</div>
50+
<!-- {{ form.as_p }} -->
3051

3152
<input class="mui-btn mui-btn--primary" type="submit" value="{% trans 'Submit' %}" />
3253
</form>
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
django
2-
django-registration
1+
django==2.2
2+
django-registration==3.0.1
33
djangorestframework
44
requests
55
django-rest-swagger
-657 Bytes
Binary file not shown.

0 commit comments

Comments
 (0)