search

zetonteam / zeton_django

Hacktoberfest
Apache License 2.0
10 stars 3 forks source link

Feature/points #28

Closed patryklomza closed 3 years ago

patryklomza commented 3 years ago

This is a starting point for our Points system.

Check updated README for a brief endpoints description.

I will add an endpoint calculating total student points in a later PR.

Waiting for your questions, and suggestions on what to improve!

visse0001 commented 3 years ago

Could you push 'makemigrations'?

Nice README description. And the tests! Too bad when I run them I get " no tests ran in 0.09s" :thinking: After hitting endpoint /api/users/points/?studentId=2 I have: 

"AssertionError at /api/users/points/
'PointResource' should either include a `serializer_class` attribute, or override the `get_serializer_class()` method."
visse0001 commented 3 years ago

Could you move pytest.ini and tests to root dir? ;)

patryklomza commented 3 years ago

Could you move pytest.ini and tests to root dir? ;)

✅ done

patryklomza commented 3 years ago

Could you push 'makemigrations'?

Nice README description. And the tests! Too bad when I run them I get " no tests ran in 0.09s" 🤔 After hitting endpoint /api/users/points/?studentId=2 I have:

"AssertionError at /api/users/points/
'PointResource' should either include a `serializer_class` attribute, or override the `get_serializer_class()` method."

✅ done