Support for custom Primary Key

laravel / cashier-mollie

MIT License

375 stars 63 forks source link

Closed ciungulete closed 3 years ago

ciungulete commented 3 years ago

Billable, instead of $owner->id it uses $owner->getKey() and in the payload we need to keep id:

{
  "owner": {
    "type": "App\\Models\\User",
    "id": 1 // this is the primary key value (steam_id, etc)
  },
  ...

if we rename payload owner -> id we can't use old payloads.

I think this is the best solution.

sandervanhooft commented 3 years ago

Agreed, good thinking!

sandervanhooft commented 3 years ago

Can you make sure it ends up in v1 too?