Add contact from address - error somewhat misleading

Bug #1390444 reported by Jonathan Cave
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Telegram app
Fix Released
High
Michał Karnicki

Bug Description

Version: 0.8.12.73

Steps to reproduce:
 * Have a contact on your phone with the phone number in local format (no country code)
 * Open telegram
 * Go to contacts list
 * Hit add contact button
 * Select "From Address Book"
 * Enter contacts selector and choose contact mentioned above and share it to telegram
 * Note the details are filled out in Add Contact form
 * Hit the tick button to accept

Actual result:
 - The dialog presents a red error message "Number is not a valid phone number"

Expected result:
 - This number *is* a valid phone number, however Telegram requires a country code to correctly identify a user

The app could may be hint that this is required rather than just displaying the current error message

Tags: cqa-verified

Related branches

Michał Karnicki (karni)
Changed in libqtelegram:
importance: Undecided → Medium
Revision history for this message
John McAleely (john.mcaleely) wrote :

Should the address book supply a default country?

Revision history for this message
John McAleely (john.mcaleely) wrote :

(or should telegram have one)

Revision history for this message
John McAleely (john.mcaleely) wrote :

Given that this has been reported by the OEM QA team, please consider raising the priority of this bug.

Michał Karnicki (karni)
Changed in libqtelegram:
importance: Medium → High
Revision history for this message
Noemí (noemi-gallego) wrote :

Any updates with this bug?

Revision history for this message
Michał Karnicki (karni) wrote :

Hello Noemi, we were planning to fix it this week. I just did that, so we'll let it go through review. Should land today, and we'll include it in a week, unless for some reason we decide to cut an intermediate release.

Changed in libqtelegram:
status: New → In Progress
assignee: nobody → Michał Karnicki (karni)
Michał Karnicki (karni)
Changed in libqtelegram:
status: In Progress → Fix Committed
Michał Karnicki (karni)
Changed in libqtelegram:
milestone: none → m14
Revision history for this message
Jonathan Cave (jocave) wrote :

Would be nice to detect automatically - but accept the improved error

tags: added: cqa-verified
Michał Karnicki (karni)
Changed in libqtelegram:
status: Fix Committed → Fix Released
Revision history for this message
Noemí (noemi-gallego) wrote :

I've tested with the new version (R19) and it shows the following message: please include the country code.
I supose this is something provisional until finding a solution right?

Revision history for this message
Michał Karnicki (karni) wrote :

Hi Noemi. We talked about this bug with another engineer, and the conclusion is:
- when adding a phone number from contacts, there's no way to determine which country is that contact from
- in case the country code is missing, we could pre-pend the logged-in user country code, so that means we assume the imported contacts are local to the user's country. I think it's a safe assumption.

Is the proposed solution what you'd expect? Please let me know here via comment. Thank you.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.