scallensc
commented
4 months ago If you aren't getting altitude, it is an API problem - both possible keys for altitude (EGM96 and WGS84) are initialised with default values of 0. These values are updated by the forPullKey listener provided by @ritirl/api, like all other variables throughout the app as per:
const useListener = () => {
const { pullKey } = keyStore.get();
useEffect(() => {
const unsubscribeListener = forPullKey(pullKey).addListener((data: IListenerProps) => {
globalStore.set((prevState) => ({
...prevState,
altitude: data.altitude,
heading: data.heading,
location: data.location,
speed: (data.speed * 3.6),
}));
});
const unsubscribeSessionListener = forPullKey(pullKey).addListener((data: ISessionListenerProps) => {
globalStore.set((prevState) => ({
...prevState,
sessionId: data.sessionId,
}));
});
return () => {
unsubscribeListener()
unsubscribeSessionListener()
};
});
};This fix serves no purpose - you will already get 0 displayed if for some reason the API is not returning altitude.
bxmatus
There is a small issue when altitude is missing. Here is corrected Altitude.tsx file which is fixing it:
import valueFormatter from "@functions/valueFormatter"; import flagStore from "@store/flagStore"; import globalStore from "@store/globalStore";
const Altitude = () => { const { showAltitude, useImperial } = flagStore.get(); const { altitude } = globalStore.get();
// Check if altitude and altitude['EGM96'] are defined if (!altitude || !altitude['EGM96']) { return null; }
const { metric, imperial } = valueFormatter('altitude', altitude['EGM96']);
return ( <div className="altitude-text" style={{ display: showAltitude ? '' : 'none' }}> Altitude: {useImperial ? imperial : metric}
export default Altitude;