mirror of
https://github.com/discourse/discourse.git
synced 2026-08-04 10:39:43 +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.
|
||
|---|---|---|
| .. | ||
| admin/assets/javascripts | ||
| app | ||
| assets | ||
| config | ||
| db | ||
| lib | ||
| public/images/upcoming_changes | ||
| spec | ||
| test/javascripts | ||
| plugin.rb | ||
| USE_CASES.md | ||
