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

New features: Cabin temperature, AC status. Climate control button.. #26

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mjuhanne
Copy link

.. and few other changes.

These changes didn't require any modification to the server protocol (Cabin temp, HVAC status and ClimateControl are already there), although I would like to have a more fine-grained AC status (i.e. Starting, Started, Stopping, Stopped, Error..) in the future.

The demo vehicle shows "-127C" at this time (I didn't yet find out where the defaults for temperatures and tire pressures are set up), but for actual vehicle it reports it correctly.

Let me know how this looks, and if OK, then I can modify the iPad storyboard also (only iPhone done now).

- Do not require tire temperature in order to show tire pressure
- Do not show temperatures as stale if cooling pump is inactive
@markwj
Copy link
Member

markwj commented Oct 14, 2019

I think it looks ok. Please add the iPad storyboard, as I have some work to do here, and it makes sense to get this merged in at the same time.

@markwj
Copy link
Member

markwj commented Oct 21, 2019

Some conflicts here, as the storyboards have been changed. Please resolve and re-submit.

@Leifur88
Copy link

Leifur88 commented Nov 12, 2019

The IOS app does not show the status of the CLIMATISATION. This is the feature to preheat or precool the cabin.. The OVSM command „xse charge hh mm“ will set the departure time for the climatisation and could preferably be SET or displayed in the app as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants