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

addresses https://github.com/aeharding/ppg.report/issues/156 #157

Merged

Conversation

dopplerreflect
Copy link
Contributor

Allows certain User settings to be overridden via URL hash parameters. userSlice initialState will attempt to fetch values from URL hash, then fall back to localStorage or defaults.

Example usage:

http://localhost:5173/29.352,-95.460#user-altitude=MSL&user-speed-unit=m/s&user-temperature-unit=%C2%B0C&user-height-unit=m&user-time-format=12-hour&user-distance-unit=km

Allows certain User settings to be overridden via URL hash parameters.
userSlice initialState will attempt to fetch values from URL hash, then
fall back to localStorage or defaults.

Example usage:
```
http://localhost:5173/29.352,-95.460#user-altitude=MSL&user-speed-unit=m/s&user-temperature-unit=%C2%B0C&user-height-unit=m&user-time-format=12-hour&user-distance-unit=km
```
@aeharding aeharding force-pushed the override-settings-with-url-params branch from 8dfcd6e to 3862123 Compare September 9, 2024 02:24
@aeharding aeharding merged commit ac15779 into aeharding:main Sep 9, 2024
1 check passed
@aeharding
Copy link
Owner

Thanks so much for the contribution! This should now be deployed in v4.3.0

@dopplerreflect
Copy link
Contributor Author

I was happy to help. I've met Caleb once, when he stopped by to visit Team Fly Halo in Salton Sea, and know a lot of people who use xcNav. I noticed you changed the || to ??, so I did a google search to learn why. Now, I'm one operator smarter. ;-) Thanks!

p.s. I've shared this app with a bunch of people in the skydiving community, and have been thinking it might benefit from less-niche name and description. It's obviously useful for way, way more than just PPG.

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