[Question] Missing fields on speed test completed event

[jdvr]

Pre-work

• I have read the docs.
• I have searched open and closed issues.
• I agree to follow this project's Code of Conduct.

Description

When configuring a webhook the payload doesn´t match the documented payload.

Expected Behavior

Get all documented fields (here)
The actual payload has less data (source)

               ->payload([
                    'result_id' => $event->result->id,
                    'site_name' => config('app.name'),
                    'isp' => $event->result->isp,
                    'ping' => $event->result->ping,
                    'download' => $event->result->downloadBits,
                    'upload' => $event->result->uploadBits,
                    'packetLoss' => $event->result->packet_loss,
                    'speedtest_url' => $event->result->result_url,
                    'url' => url('/admin/results'),
                ])

Steps to Reproduce

1. Set a webhook as notification
2. Run a test

Deployment Environment

Docker Compose

Application Information

root@976dffe541a6:/# php artisan about --json
Could not open input file: artisan

What browser(s) are you seeing the problem on?

Firefox

Logs

[jdvr]

I can submit a PR if you agree 👍

[svenvg93]

That was my error, the PR to update the docs was merged, but the PR to update the code was not as we shifted focus to v2. I will discuss with Alex what we want tot do, either we will update the docs and it the fields in v2 or I will update add the fields in an PR.

[jdvr]

Thanks @svenvg93