All users were logged out of Bugzilla on October 13th, 2018

Reference Workloads - SMS generation doesn't use contact numbers

RESOLVED FIXED

Status

RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: jhylands, Assigned: jhylands)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
In reference workloads, the dialer history uses contact numbers for a certain percentage of the entries. These numbers show up as the contact name rather than the phone number. SMS messages use random phone numbers, but they should use contact numbers for a certain percentage the same way dialer history does.
(Assignee)

Updated

6 years ago
Assignee: nobody → jhylands
(Assignee)

Comment 1

6 years ago
Created attachment 738941 [details]
pull request - Changed SMS generation to use contact numbers
Attachment #738941 - Flags: review?(anygregor)
Attachment #738941 - Flags: review?(anygregor) → review+
John, can you merge this? It doesn't let me merge by just pressing the button.
Flags: needinfo?(jhford)
master: 33f00eb7527d37b5f5b28fba0d5e712bea445c58

Jon, I checked that the sha1sum of the conflicting file on your branch was equal to the one in the above merge commit.  Can you please double check this and mark the bug as fixed if everything is sane?

John
Flags: needinfo?(jhford) → needinfo?(jhylands)
Spoke with Jon, they are the correct files.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Comment 5

6 years ago
This was resolved verbally with jhford in Madrid...
Flags: needinfo?(jhylands)
You need to log in before you can comment on or make changes to this bug.