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_controller_spec.rb | ||
| credentials_controller_spec.rb | ||
| data_table_columns_controller_spec.rb | ||
| data_table_rows_controller_spec.rb | ||
| data_tables_controller_spec.rb | ||
| dynamic_node_parameters_controller_spec.rb | ||
| executions_controller_spec.rb | ||
| expressions_controller_spec.rb | ||
| filter_options_controller_spec.rb | ||
| form_test_sessions_controller_spec.rb | ||
| forms_controller_spec.rb | ||
| node_types_controller_spec.rb | ||
| post_button_controller_spec.rb | ||
| stats_controller_spec.rb | ||
| step_executions_controller_spec.rb | ||
| templates_controller_spec.rb | ||
| topic_admin_button_controller_spec.rb | ||
| variables_controller_spec.rb | ||
| webhook_test_listeners_controller_spec.rb | ||
| webhooks_controller_spec.rb | ||
| workflow_tags_controller_spec.rb | ||
| workflow_versions_controller_spec.rb | ||
| workflows_controller_spec.rb | ||
