Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to Tabulator 6.3.0 #921

Merged
merged 1 commit into from
Oct 14, 2024
Merged

Conversation

argilo
Copy link
Contributor

@argilo argilo commented Oct 13, 2024

Fixes #915.

Updating Tabulator fixes the issue with the telemetry table (and map) displaying incorrectly in Firefox. I chose the latest available release, 6.3.0. A number of API changes have been made, so I updated the Auto-RX code where needed.

I've tested the Historical page, but haven't yet had a chance to verify that the telemetry table works correctly during a flight. (I'll have a chance in an hour or two.)

@argilo
Copy link
Contributor Author

argilo commented Oct 13, 2024

It's probably easiest to review the code changes while ignoring differences in white space: https:/projecthorus/radiosonde_auto_rx/pull/921/files?w=1

@argilo
Copy link
Contributor Author

argilo commented Oct 13, 2024

I'm tracking a flight now, and the telemetry table looks to be working as expected.

@argilo
Copy link
Contributor Author

argilo commented Oct 14, 2024

I noticed that the source map for Tabulator (which I had added) was rather large, so I took it out to avoid bloating Auto-RX. If anyone needs it for debugging purposes, they can always download their own copy.

@argilo
Copy link
Contributor Author

argilo commented Oct 14, 2024

I also noticed that I forgot to update jquery_wrapper.min.js, which is a part of Tabulator. But it turns out Auto-RX doesn't use it, so I'll remove it instead.

@darksidelemm darksidelemm merged commit e738039 into projecthorus:testing Oct 14, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants