weeman weeman
  • Joined on 2021-03-21
weeman commented on pull request kompetenzinventar/ki-backend#5 2021-06-21 17:40:53 +02:00
auth.py: Make usage of exceptions instead of if-else-blocks

Siehe #7 für Ausgaben.

weeman created pull request kompetenzinventar/ki-backend#7 2021-06-21 17:40:23 +02:00
Logging
weeman commented on pull request kompetenzinventar/ki-backend#6 2021-06-21 17:38:17 +02:00
Add yapf to Pipenv environment

Passt, danke für die Erläuterung.

weeman pushed to main at kompetenzinventar/ki-backend 2021-06-21 17:38:06 +02:00
ea7b6391c1 Merge pull request 'Add yapf to Pipenv environment' (#6) from add-yapf into main
cbf3002b93 Reformat source code
6d4f933585 Add yapf to Pipfile
1390dfa8e6 Add usage of yapf to README
Compare 4 commits »
weeman merged pull request kompetenzinventar/ki-backend#6 2021-06-21 17:38:06 +02:00
Add yapf to Pipenv environment
weeman pushed to feature-logging at kompetenzinventar/ki-backend 2021-06-21 17:35:40 +02:00
3dcba71a6d add logging
weeman created branch feature-logging in kompetenzinventar/ki-backend 2021-06-21 17:35:40 +02:00
weeman commented on issue kompetenzinventar/ki-backend#4 2021-06-21 17:12:50 +02:00
Creating a new token failes if there is an old present

sqlalchemy.exc.IntegrityError: (sqlite3.IntegrityError) NOT NULL constraint failed: token.user_id

Liest sich jetzt eher so als gäbe es den Benutzer nicht.

Gut für die Nachvollziehbarkeit…

weeman commented on pull request kompetenzinventar/ki-backend#5 2021-06-21 17:10:32 +02:00
auth.py: Make usage of exceptions instead of if-else-blocks

Die Prüfung, ob der Benutzer existiert ist verschwunden oder übersehe ich was?

weeman commented on pull request kompetenzinventar/ki-backend#5 2021-06-21 17:08:36 +02:00
auth.py: Make usage of exceptions instead of if-else-blocks

Hier würde ich dann eine 500er Antwort zurückgeben.

weeman commented on pull request kompetenzinventar/ki-backend#5 2021-06-21 17:07:50 +02:00
auth.py: Make usage of exceptions instead of if-else-blocks

Ich bau die Tage mal logging ein

weeman suggested changes for kompetenzinventar/ki-backend#6 2021-06-21 17:06:40 +02:00
Add yapf to Pipenv environment
weeman commented on pull request kompetenzinventar/ki-backend#6 2021-06-21 17:06:40 +02:00
Add yapf to Pipenv environment

Formating gibt es nicht. Entsprechend dem Rest der Datei Formatierung oder Formatieren?

weeman commented on pull request kompetenzinventar/ki-backend#6 2021-06-21 17:06:15 +02:00
Add yapf to Pipenv environment

Find ich gut. Ich würde auch später mal sehen Git-Hooks mit reinzunehmen.

Eine Frage hätte ich: Wenn yapf PIP8 formatiert und flake8 PIP8 validiert, wieso war es vorher schon gültig? Das…

weeman pushed to main at kompetenzinventar/ki-backend 2021-06-20 20:21:52 +02:00
f20db25f74 extend profile update test
weeman pushed to main at kompetenzinventar/ki-backend 2021-06-20 20:13:24 +02:00
ace51851de implement update profile endpoint
weeman commented on issue kompetenzinventar/ki-doku#10 2021-06-20 19:28:11 +02:00
[backend] Drone Setup

Offen ist noch ein eigenes Test-Image damit das nicht jedes mal neu gebaut werden muss. Bin dran. Die Admins wollen noch einen PGP Schlüssel.

weeman commented on pull request kompetenzinventar/ki-doku#11 2021-06-20 19:26:54 +02:00
data model adapted to split between user and profile

Es ist vollbracht: 5c21e4c9b6