Skip to content

Comments

Add explicit deal type support for schedules#623

Open
anjor wants to merge 3 commits intomainfrom
feat/schedule-explicit-dealtype
Open

Add explicit deal type support for schedules#623
anjor wants to merge 3 commits intomainfrom
feat/schedule-explicit-dealtype

Conversation

@anjor
Copy link
Collaborator

@anjor anjor commented Feb 20, 2026

Summary

  • add explicit dealType on schedule model (market or pdp)
  • accept and validate dealType in schedule create/update handlers
  • add CLI flags:
    • singularity deal schedule create --deal-type
    • singularity deal schedule update --deal-type
  • make dealpusher honor schedule.dealType in default resolution (while keeping existing resolver override behavior)
  • add tests for invalid deal type rejection and successful persistence

Validation

  • go test ./handler/deal/schedule
  • go test ./service/dealpusher
  • go test ./cmd

@anjor anjor force-pushed the feat/schedule-explicit-dealtype branch 2 times, most recently from 33492a6 to fd2b1dc Compare February 24, 2026 14:19
@anjor anjor requested a review from parkan February 24, 2026 14:31
@anjor anjor force-pushed the feat/schedule-explicit-dealtype branch from fd2b1dc to f38d79b Compare February 24, 2026 16:36
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.

1 participant