mirror of
https://github.com/discourse/discourse.git
synced 2026-08-05 16:23:34 +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.
|
||
|---|---|---|
| .. | ||
| controllers/discourse_workflows | ||
| jobs | ||
| models/discourse_workflows | ||
| serializers/discourse_workflows | ||
| services/discourse_workflows | ||
