Skip to content

Conversation

@stark-cybertech
Copy link
Collaborator

@stark-cybertech stark-cybertech commented Jul 2, 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

Merges #61 into main

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

stark-cybertech and others added 30 commits June 30, 2025 12:12
Marks all fields that aren't explicitly optional as required.
stark-cybertech and others added 23 commits July 1, 2025 14:34
Note: Algorithm is unimplemented for now.
Add 409 return value to register match request
Tests currently fail. Will be fixed in next commit
Moving getMatchStatus(UUID groupID) into GroupService allows removing the dependency on MatchService.
@stark-cybertech stark-cybertech self-assigned this Jul 2, 2025
@stark-cybertech stark-cybertech merged commit 7fe8247 into main Jul 2, 2025
9 checks passed
@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.

2 participants