Skip to content

feat: allow to pass SlotSearchType to find-appointment command#14

Merged
rafsaf merged 2 commits intorafsaf:mainfrom
dawidgora:feat/allow-to-pass-slot-search-type
Nov 14, 2025
Merged

feat: allow to pass SlotSearchType to find-appointment command#14
rafsaf merged 2 commits intorafsaf:mainfrom
dawidgora:feat/allow-to-pass-slot-search-type

Conversation

@dawidgora
Copy link
Contributor

Changes

  • Add SlotSearchType support to find-appointment, including a --slot-search-type/-S flag and propagation
    through AppointmentFinder.
  • Introduce DEFAULT_SLOT_SEARCH_TYPE and a parser helper so the API payload can accept either numeric or
    string slot types.
  • Update unit tests to assert the new parameter is sent to the API and that the CLI wiring handles both
    explicit values and the default.

@dawidgora dawidgora force-pushed the feat/allow-to-pass-slot-search-type branch from 8982a14 to 89cfa05 Compare November 6, 2025 09:02
@dawidgora
Copy link
Contributor Author

FYI @rafsaf

@rafsaf
Copy link
Owner

rafsaf commented Nov 13, 2025

Thanks a lot, looks fine at first glance but it will need another look today or tomorrow

@rafsaf rafsaf merged commit f7546cc into rafsaf:main Nov 14, 2025
4 checks passed
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.

5 participants