Closed Bug 936648 Opened 11 years ago Closed 11 years ago

[Messages][Drafts] Draft display and interaction within a thread

Categories

(Firefox OS Graveyard :: Gaia::SMS, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
1.3 Sprint 5 - 11/22

People

(Reporter: evhan55, Assigned: protonk)

References

Details

Attachments

(1 file)

46 bytes, text/x-github-pull-request
protonk
: review?
rwaldron
Details | Review
Display list of drafts within a thread (in a Drafts tab) Long press of a draft gives option to edit or delete
Assignee: nobody → achyland
Target Milestone: --- → 1.3 Sprint 5 - 11/22
Component: Gaia → Gaia::SMS
Attached file Github PR
Cherry-picks rick's patch from Bug 935177
Attachment #8338791 - Flags: review?(waldron.rick)
1. needs to loop through recipients, do a contact lookup by phone number and call ThreadUI.recipients.add({ ... }) with the appropriate data 2. Recall for "#new" (with or without recipients) needs to look at MessageManager.draft for an associated draft to populate from 3. "#thread=id" can find its draft at `Drafts.byThreadId(threadId).latest` 4. Upon sending, the draft object needs to be removed and the storage synced
Blocks: 946995
Rick, we need to make it apparent that this patch has been r+ on the bug (I see that you reviewed it with the commit log). Can you attach the PR here and put r+?
Flags: needinfo?(waldron.rick)
Depends on: 947211
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: needinfo?(waldron.rick)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: