mirror of
https://github.com/discourse/discourse.git
synced 2026-08-14 13:58:53 +08:00
ed24248e30Previously, the workflows admin showed "0 Bytes" for every data table
and the executions list never loaded more than the first page, because
`Service::Runner` only fills **required** keyword arguments of
`on_success` blocks from the service result — defaulted kwargs like
`table_sizes: {}` and `load_more_url: nil` silently kept their defaults,
dropping the values the services computed.
This change declares both keyword arguments as required, and moves
`Execution::List`'s `load_more_url` step out of `only_if(:has_more)`
(guarding inside the compute method instead, like `Workflow::List`) so
the context key always exists and the required kwarg can never raise.
|
||
|---|---|---|
| .. | ||
| discourse_workflows | ||
