mirror of
https://github.com/discourse/discourse.git
synced 2026-08-05 20:29:55 +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.
|
||
|---|---|---|
| .. | ||
| ai_authoring | ||
| concerns | ||
| credential | ||
| data_table | ||
| data_table_column | ||
| data_table_row | ||
| execution | ||
| expression | ||
| form | ||
| form_test_session | ||
| modal | ||
| node_type | ||
| policy | ||
| stats | ||
| template | ||
| variable | ||
| webhook | ||
| webhook_test_listener | ||
| workflow | ||
| workflow_tag | ||
| workflow_version | ||
