Profilsuche nach Nickname #41

Merged
weeman merged 1 commits from feature-nickname-search into main 2021-07-26 19:40:34 +02:00
Owner
  • Suche nach Nickname
  • Den Test mal richtig gemacht - lol

closes #37

- Suche nach Nickname - Den Test mal richtig gemacht - lol closes #37
weeman added 1 commit 2021-07-12 21:15:35 +02:00
continuous-integration/drone/push Build is passing Details
continuous-integration/drone/pr Build is passing Details
f2ea02f7e3
add search by nick
frlan reviewed 2021-07-26 18:26:59 +02:00
@ -22,0 +22,4 @@
if "nickname" in request.args:
nickname = request.args.get("nickname")
query = query.filter(Profile.nickname.like(f"%{nickname}%"))
Owner

Possible SQL injection? Needs to be double checked

Possible SQL injection? Needs to be double checked
Author
Owner

Laut Doku escaped SQLAlchemy das automagisch.
Aber lass einen Test reinpacken.

Laut Doku escaped SQLAlchemy das automagisch. Aber lass einen Test reinpacken.
Author
Owner

Test eingefügt

Test eingefügt
weeman marked this conversation as resolved
weeman added 1 commit 2021-07-26 19:30:58 +02:00
continuous-integration/drone/push Build is passing Details
continuous-integration/drone/pr Build is passing Details
840cf8830d
add sql special chars test
weeman force-pushed feature-nickname-search from 840cf8830d to 76e95311f0 2021-07-26 19:31:36 +02:00 Compare
weeman merged commit ea57f326b7 into main 2021-07-26 19:40:33 +02:00
weeman deleted branch feature-nickname-search 2021-07-26 19:40:38 +02:00
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: kompetenzinventar/ki-backend#41
No description provided.