Skip to content

Conversation

@stark-cybertech
Copy link
Collaborator

@stark-cybertech stark-cybertech commented Jul 1, 2025

Type of Change

  • πŸ› Bug fix
  • ✨ New feature
  • ♻️ Code refactor
  • πŸ“ Documentation update
  • πŸš€ Performance improvement
  • βœ… Test addition/update
  • 🧹 Chore/maintenance
  • πŸ“ API Spec update
  • Other (please describe):

Changes

  • Implement logic for MatchingService
  • implement services for handling database entries
  • implement services for handling matching logic
  • implement REST controller
  • implement skeleton for match algorithm
  • Update API to v2.1.6
  • 2.1.2: set required flags
  • 2.1.3: bugfix for collections
  • 2.1.4: remove UUID field from MatchRequestUpdate
  • 2.1.5: bugfix incorrect operation names
  • 2.1.6: add 409 response to register match requests
  • Add basic testing sturcture
  • see Feature/matching service testsΒ #63

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have updated the API spec to reflect any changes my code makes

Related Issues

Fixes #
Related to #

Additional Notes

This initial implementation of the Matching-microservice contains most of the basic logic, albeit still untested pending Gateway-service's completion.

Much of the code here comes from the automatic generation of client templates through openapi-generator-cli. All code in */generated-client/* is auto-generated and can safely be ignored.

This is a WiP, and not all features are working.

Complete

Services

  • GroupService
  • MatchingService
  • MatchRequestService
  • TimeslotService

Algorithm

  • MatchingSolution
  • MatchingSolutionBlock

Controler

  • MatchingController

Skeleton / Untested

Services

  • ConversationStarterService
  • MatchingService

Client

  • UserClient

No Implementation

Services

  • Scheduling Service

Client

  • GenAiClient

Algorithm

  • MatchingAlgorithm

EnricoBausenhart and others added 30 commits June 16, 2025 08:46
Marks all fields that aren't explicitly optional as required.
Copy link
Collaborator

@a-kori a-kori left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@stark-cybertech stark-cybertech changed the title Feature/matching service Feature/matching-service WiP Jul 2, 2025
@stark-cybertech stark-cybertech merged commit 5a3421e into feature/user-service Jul 2, 2025
9 checks passed
@stark-cybertech stark-cybertech mentioned this pull request Jul 2, 2025
20 tasks
stark-cybertech added a commit that referenced this pull request Jul 2, 2025
This merges the changes listed in #61 into main
@a-kori a-kori deleted the feature/matching-service branch July 14, 2025 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants