Phone number highlighting is not localized
Bug #1376803 reported by
Víctor R. Ruiz
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Canonical System Image |
Fix Released
|
High
|
Bill Filler | ||
| messaging-app |
Fix Released
|
High
|
Alexandre Abreu | ||
| messaging-app (Ubuntu) |
Fix Released
|
High
|
Alexandre Abreu | ||
| messaging-app (Ubuntu RTM) |
Fix Released
|
High
|
Gustavo Pichorim Boiko | ||
| telephony-service (Ubuntu) |
Fix Released
|
High
|
Unassigned | ||
| telephony-service (Ubuntu RTM) |
In Progress
|
Undecided
|
Gustavo Pichorim Boiko | ||
Bug Description
Test plan has a case to "Receive an sms that has a phone number in it". Test case passes, but only if the number has a US (I guess) format, that is 617-688-0034. Spanish phone numbers like "666.777.888" or "666554433" aren't highlighted.
Related branches
lp://staging/~abreu-alexandre/messaging-app/phone-number-detection-msg-bubbles
- Tiago Salem Herrmann (community): Approve
- PS Jenkins bot: Needs Fixing (continuous-integration)
-
Diff: 66 lines (+20/-2)2 files modifieddebian/control (+2/-0)
src/qml/MessageBubble.qml (+18/-2)
lp://staging/~abreu-alexandre/telephony-service/expose-phone-number-matcher
- Renato Araujo Oliveira Filho (community): Approve
- PS Jenkins bot: Approve (continuous-integration)
-
Diff: 94 lines (+59/-0)3 files modifiedUbuntu/Telephony/PhoneNumber/phoneutils.cpp (+20/-0)
Ubuntu/Telephony/PhoneNumber/phoneutils.h (+2/-0)
Ubuntu/Telephony/tests/tst_PhoneNumberPhoneUtils.qml (+37/-0)
lp://staging/~boiko/messaging-app/rtm-fix_1376803
- Ubuntu Phablet Team: Pending requested
-
Diff: 66 lines (+20/-2)2 files modifieddebian/control (+2/-0)
src/qml/MessageBubble.qml (+18/-2)
lp://staging/~boiko/telephony-service/rtm-fix_1376803
- Ubuntu Phablet Team: Pending requested
-
Diff: 94 lines (+59/-0)3 files modifiedUbuntu/Telephony/PhoneNumber/phoneutils.cpp (+20/-0)
Ubuntu/Telephony/PhoneNumber/phoneutils.h (+2/-0)
Ubuntu/Telephony/tests/tst_PhoneNumberPhoneUtils.qml (+37/-0)
| summary: |
- Phone number highlighting issues + Phone number highlighting is not localized |
Revision history for this message
| Bill Filler (bfiller) wrote | #1 |
| Changed in messaging-app: | |
| status: | New → Incomplete |
Revision history for this message
| Víctor R. Ruiz (vrruiz) wrote | #2 |
Revision history for this message
| Alexandre Abreu (abreu-alexandre) wrote | #3 |
Revision history for this message
| Alexandre Abreu (abreu-alexandre) wrote | #4 |
| Changed in messaging-app: | |
| status: | Incomplete → In Progress |
| importance: | Undecided → High |
| assignee: | nobody → Alexandre Abreu (abreu-alexandre) |
| Changed in messaging-app (Ubuntu): | |
| importance: | Undecided → High |
| Changed in messaging-app (Ubuntu RTM): | |
| importance: | Undecided → High |
| Changed in messaging-app (Ubuntu): | |
| status: | New → In Progress |
| assignee: | nobody → Alexandre Abreu (abreu-alexandre) |
| Changed in telephony-service (Ubuntu): | |
| importance: | Undecided → High |
| tags: | added: ww03 |
| tags: |
added: ww07 removed: ww03 |
| Changed in messaging-app (Ubuntu): | |
| status: | In Progress → Fix Released |
| Changed in telephony-service (Ubuntu): | |
| status: | New → Fix Released |
| Changed in messaging-app (Ubuntu RTM): | |
| status: | New → In Progress |
| assignee: | nobody → Gustavo Pichorim Boiko (boiko) |
| Changed in telephony-service (Ubuntu RTM): | |
| status: | New → In Progress |
| assignee: | nobody → Gustavo Pichorim Boiko (boiko) |
| Changed in canonical-devices-system-image: | |
| importance: | Undecided → High |
| milestone: | none → ww07-2015 |
| status: | New → In Progress |
| Changed in canonical-devices-system-image: | |
| assignee: | nobody → Bill Filler (bfiller) |
| milestone: | ww07-2015 → ww09-2015 |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #5 |
| Changed in messaging-app (Ubuntu RTM): | |
| status: | In Progress → Fix Released |
| Changed in canonical-devices-system-image: | |
| status: | In Progress → Fix Released |
| Changed in messaging-app: | |
| status: | In Progress → Fix Released |
To post a comment you must log in.
I don't understand what is supposed to be highlighted? What is the use case that is not working?