Dielee / volvo2mqtt

Home Assistant addon for connecting AAOS Volvos
MIT License
125 stars 25 forks source link

Phone number as user name not accepted #170

Closed larionof closed 3 months ago

larionof commented 4 months ago

Describe the bug The addon only accepts emails as user name though many of us have been forced to use phone number as user name.

Screenshots:
Failed to save add-on configuration, does not match regular expression ^([a-zA-Z0-9_.+-]+@[a-zA-Z0-9-]+.[a-zA-Z0-9-.]+)|(+\d{5,20})$. Got {'updateInterval': 300, 'babelLocale': 'FI', 'TZ': 'Europe/Helsinki', 'debug': False, 'disable_logging': False, 'mqtt': {'broker': 'auto_broker', 'port': 'auto_port', 'username': 'auto_user', 'password': 'auto_password'}, 'volvoData': {'username': 355555555, 'password': 'password', 'vin': '', 'vccapikey': [None], 'odometerMultiplier': 1, 'averageSpeedDivider': 1, 'averageFuelConsumptionMultiplier': 1}.

Version info:
The version you are using and the method you are using (Docker container/HA Addon/Cloned repo)

Logs:
Enable the debug option and upload your logfile here.
If you are using the HA Addon the log location is \\<Your HA Host IP>\addons\volvo2mqtt\log\volvo2mqtt.log

Dielee commented 4 months ago

Please send me your phone number, containing some zeros to make it invalid.

larionof commented 4 months ago

An example phone number showing the format we use for P8: +358405023410. The shortest number though is +358400500 or alike

Dielee commented 4 months ago

So the regex works for your phone number. There is something else wrong. Please send me a screenshot from your config.

github-actions[bot] commented 4 months ago

This issue is stale because it has been open 15 days with no activity. Remove stale label or comment or this will be closed in 5 days.

larionof commented 4 months ago

I do apologise not getting back to this earlier. Here's my data: github-virhe

Dielee commented 4 months ago

How long is your phone number, including the "+" in front ?

larionof commented 4 months ago

Most common phone number format is +358 123 456 7890, so 1+3+10= 14

Dielee commented 4 months ago

So you are using whitespaces in your phone number ?

larionof commented 4 months ago

No. I wrote the format out without thinking, sorry for that( acc. grammar it is written as above but obviously wherever needed the all 14 marks( numbers, letters, plus sign) are without spaces).

Dielee commented 4 months ago

So, i can't help you without beeing able to see your full config. Are you able to share me your whole config via PM ?

larionof commented 4 months ago

Understood but best I can provide is an example of the settings used( love the work you've done to get the interface to HA and if majority get it to work with email as username, that's acceptable). What I will try next is to add an account to the car with an email as username instead of the phone number. I've not bothered to make secondary accounts yet as they're not required to have phones connected

github-virhe

github-actions[bot] commented 3 months ago

This issue is stale because it has been open 15 days with no activity. Remove stale label or comment or this will be closed in 5 days.

github-actions[bot] commented 3 months ago

This issue was closed because it has been stalled for 5 days with no activity.