ee308c637c
This PR fixes 2 issues that were picked up by users for the Scheduled Data Explorer Report automation script and a couple of small improvements to better match the format of manual data explorer query results. The first issue is that within result_to_markdown the colrender contains null values, and we are currently checking length (previously treated as a packed array but it is actually a sparse array). Therefore we can check if the current index of the array is null rather than checking the size of the array. The second issue addresses the blank query_params field. When the data explorer script does not require any params to be passed in via the automation script then it will have a nil value, however it should be defaulted to {} within the plugin. To improve formatting the markdown table for PMs is now aligned to left and where values are substituted (for example user_id becomes username) we then include the id within brackets, for example: user_id becomes username (user_id) |
||
---|---|---|
.. | ||
automation | ||
fabricators | ||
integration | ||
jobs/scheduled | ||
lib/data_explorer | ||
requests | ||
system | ||
tasks | ||
data_explorer_spec.rb | ||
guardian_spec.rb | ||
report_generator_spec.rb | ||
result_to_markdown_spec.rb |