[Bug] Fix incorrectly formatted json data

[alexjustesen]

📃 Description

This PR fixes incorrectly formatted json data in a Result's data column.

• closes Incorrectly formatted json in results data column #310
• closes Include public IP in results + add option to filter per public IP #513
• closes Hide unsuccessful results on dashboard #993
• closes Timeout opening Resultspage #1060
• closes [FEATURE REQUEST] CSV export including Public IP address #1127

FAQ

• Q: Where the F*** did my data go?
  • It was copied to a new table called results_bad_json for safe keeping.
• Q: Will I lose my old results data?
  • No, even after the data is migrated to the new results table results_bad_json will remain in the database unless you decide to manually drop the table.
• Q: Why were my scheduled speedtests disabled?
  • This fix is so important it requires your attention, once you've confirmed a successful data migration you can enable your scheduled speedtests again.

Process

Database Migration (automated)

The following steps will occur automatically as part of the 0.16.0 version.

1. Rename results to results_bak_bad_json so theirs a copy of existing data.
2. Migrate new results table with an updated DDL.
3. If results already exist (aka existing install).
   1. Disable the existing speedtest schedule.
   2. Send a database notification to each admin user that manual action is required.

Data Migration (manual)

The following steps will be required should users want to move their data over once they've upgraded to v0.16.0. NOTE: this only applies to existing installs with existing results data.

1. Adds a data migration job to the job queue.
2. Copy each record from results_bad_json to results with the new DDL format.

To-dos

• Test manual speedtests
• Test scheduled speedtests
• Test import migration
  • Make sure it doesn't trigger any writes to influxdb
• Test influxdb integration
• Test admin dashboard
• Test public dashboard
• Test results table
• Test home lab API endpoint
• After data migration show / hide option and/or remove table, need to clean up this data or have some indicator for a successful migration.

🪵 Changelog

➕ Added

• jitter and service to view result form.
• MigrateBadJsonResults action to move result data from results_bad_json to results.
• Result model attributes to get additional information from data column.
  • download_jitter
  • ip_address
  • isp
  • packet_loss
  • ping_jitter
  • result_url
  • server_host
  • server_id
  • server_name
  • upload_jitter
• service which preps for supporting multiple speedtest services (other than Ookla).
• ResultStatus enum to strictly set the result's status.
• ip_address result table filter
• status result table filter
• updated_at back to the Result model.

✏️ Changed

• chart and stat widgets only return Completed speedtests.
• completely refactored Results table to show additional columns and support Number helper.
• formatForInfluxDB2() method to use model attributes.
• view result form, moved server info to right column.

🔧 Fixed

• writing incorrectly formatted JSON strings in data column in the results table.

🗑️ Removed

• $fillable = [] Result model attribute in favor of $guarded = []
• getJitterData() method from Result model, no longer needed.