kompetenzinventar/ki-backend
5
6
Fork 7
Code Issues Pull Requests 13 Packages Projects Releases 1 Activity

Klarnamen in der Suche berücksichtigen #68

Merged
jaschop merged 3 commits from zeitschlag/ki-backend:feature/48-search-address-name into main 2023-10-19 14:28:59 +02:00
Conversation 1 Commits 3 Files Changed 3 +29 -11
zeitschlag commented 2023-04-20 17:36:29 +02:00
Member
Copy Link

Backend-Teil für kompetenzinventar/ki-doku#48

Siehe auch kompetenzinventar/ki-frontend#80

Backend-Teil für [kompetenzinventar/ki-doku#48](https://git.wtf-eg.de/kompetenzinventar/ki-doku/issues/48) Siehe auch [kompetenzinventar/ki-frontend#80](https://git.wtf-eg.de/kompetenzinventar/ki-frontend/pulls/80)
zeitschlag added 1 commit 2023-04-20 17:36:29 +02:00
zeitschlag changed title from Klarnamen in der Suche berücksichtigen to WIP: Klarnamen in der Suche berücksichtigen 2023-04-20 17:49:52 +02:00
zeitschlag force-pushed feature/48-search-address-name from c24dbd3ebe to b5053e178f 2023-04-20 18:11:22 +02:00 Compare
zeitschlag changed title from WIP: Klarnamen in der Suche berücksichtigen to Klarnamen in der Suche berücksichtigen 2023-04-20 18:12:15 +02:00
zeitschlag changed title from Klarnamen in der Suche berücksichtigen to [BE] Klarnamen in der Suche berücksichtigen 2023-04-20 18:15:19 +02:00
zeitschlag changed title from [BE] Klarnamen in der Suche berücksichtigen to Klarnamen in der Suche berücksichtigen 2023-04-20 18:15:53 +02:00
zeitschlag added 1 commit 2023-04-20 18:25:48 +02:00
jaschop requested changes 2023-10-13 21:27:45 +02:00
jaschop left a comment
Member
Copy Link

EIn Manko beim seeden gefunden.
Branch ist etwas out of date, aber nach einem 'git merge main' hat bei mir alles funktioniert.

Bitte seed anpassen, mergen, und neuen PR aufmachen.

EIn Manko beim seeden gefunden. Branch ist etwas out of date, aber nach einem 'git merge main' hat bei mir alles funktioniert. Bitte seed anpassen, mergen, und neuen PR aufmachen.
ki/actions/seed.py Outdated
@ -156,0 +156,4 @@
freetext="1001010010111!!!",
skills=[(Skill.skill_id_php, 5)])
dieters_address = Address(name="Friedrich Witzig", profile=dieter)
jaschop commented 2023-10-13 21:22:02 +02:00
Member
Copy Link

Diese Verwendung bricht das existierende Pattern von seed_user und vermischt Abstraktionslevel.
Die Adresse könnte wie die Skills als Parameter übergeben werden.

Diese Verwendung bricht das existierende Pattern von `seed_user` und vermischt Abstraktionslevel. Die Adresse könnte wie die Skills als Parameter übergeben werden.
zeitschlag commented 2023-10-14 21:12:45 +02:00
Author
Member
Copy Link

Wie besprochen sollten wir die Erstellung von Profilen refactoren.

Wie besprochen sollten wir die Erstellung von Profilen refactoren.
zeitschlag marked this conversation as resolved
zeitschlag force-pushed feature/48-search-address-name from a9390a35aa to d7f4acf251 2023-10-14 21:01:17 +02:00 Compare
zeitschlag requested review from jaschop 2023-10-14 21:13:38 +02:00
jaschop commented 2023-10-19 14:28:49 +02:00
Member
Copy Link

Nach weiterer Review waren die Quality Issues nicht von diesem spezifischem Commit verursacht.

Refactoring wird separat unter kompetenzinventar/ki-doku#56 getrackt.

Nach weiterer Review waren die Quality Issues nicht von diesem spezifischem Commit verursacht. Refactoring wird separat unter kompetenzinventar/ki-doku#56 getrackt.
👍 1 🎉 1 🚀 1
jaschop merged commit d7f4acf251 into main 2023-10-19 14:28:59 +02:00
zeitschlag deleted branch feature/48-search-address-name 2023-10-19 14:30:50 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
jaschop
Labels
Clear labels   
bug

Something is not working

  
doing

   
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
ready

Bereit für Release

  
review

   
security

   
wontfix

This won't be fixed

No Label
bug
doing
duplicate
enhancement
help wanted
invalid
question
ready
review
security
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
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#68
No description provided.
Delete Branch "zeitschlag/ki-backend:feature/48-search-address-name"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?