Add speedtest result bytes to results table

[ilteoood]

📃 Description

Closes #2109

📖 Documentation

I think it is not needed

🪵 Changelog

➕ Added

• storage and display of amount of bytes downloaded
• storage and display of amount of bytes uploaded

📷 Screenshots

[ilteoood]

NOTES:

• I'm not able to make the csv export to work. The column is generated but is always empty. I'm out of ideas, any help here is appreciated.
• My PHP is rusty, sorry if this is a mess for you

[svenvg93]

• I'm not able to make the csv export to work. The column is generated but is always empty. I'm out of ideas, any help here is appreciated.

I checkend the export for you, it all works for me

Did you restart the queue:work in between changes? I had it in the past that a restart was needed for code changes to be picked up 😅 .

[ilteoood]

• I'm not able to make the csv export to work. The column is generated but is always empty. I'm out of ideas, any help here is appreciated.

I checkend the export for you, it all works for me

Did you restart the queue:work in between changes? I had it in the past that a restart was needed for code changes to be picked up 😅 .

Nope I haven't, this could explain what happened. Thank you so much!

[alexjustesen]

@ilteoood thanks for the PR!

I'm going to leave a review on individual files with any changes but can you please remove any UI changes (i.e. the dashboard charts). Reason for this is I'm actively researching replacing the dashboard and don't want to add tech debt/scope at the moment.

I also attached #2109 to #2146 and as I think through that feature I might come back to your PR with a couple changes.

[ilteoood]

@ilteoood thanks for the PR!

I'm going to leave a review on individual files with any changes but can you please remove any UI changes (i.e. the dashboard charts). Reason for this is I'm actively researching replacing the dashboard and don't want to add tech debt/scope at the moment.

I also attached #2109 to #2146 and as I think through that feature I might come back to your PR with a couple changes.

With UI changes you mean also the references to the result and export tables?

[alexjustesen]

With UI changes you mean also the references to the result and export tables?

Exports and results table we'll want, just the charts we dont.

[ilteoood]

@alexjustesen everything should be in place with what you asked.
Thank you so much for the time you dedicated reviewing this.

[alexjustesen]

Thanks for the changes! One more round of changes after finally having a moment to look at the PR.

[alexjustesen]

One other quick note, you'll notice I changed the PR title and base branch. Going to roll this into the larger #2146 feature.

[radh21301]

Hello. Just wanted to double check. Is this feature present in the latest release? I am not able to see the column for
Upload/Download Bytes.
FYI: I am using linuxserver image.

[alexjustesen]

Hello. Just wanted to double check. Is this feature present in the latest release? I am not able to see the column for
Upload/Download Bytes.
FYI: I am using linuxserver image.

This got merged into a feature branch that hasn't been released yet.

[ilteoood]

@alexjustesen are you still planning to complete that feature?
It's been a while since my contribution has been merged and I would like to start using it.

[alexjustesen]

@alexjustesen are you still planning to complete that feature?
It's been a while since my contribution has been merged and I would like to start using it.

It's still planned, I'm on parental leave at the moment so I'm not going to touch anything that's not a big fix for the next few weeks.

[ilteoood]

@alexjustesen are you still planning to complete that feature?
It's been a while since my contribution has been merged and I would like to start using it.

It's still planned, I'm on parental leave at the moment so I'm not going to touch anything that's not a big fix for the next few weeks.

Wow, that's a great achievement! Congratulations! 👏