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)
Tracking
(Not tracked)
RESOLVED
FIXED
1.3 Sprint 5 - 11/22
People
(Reporter: evhan55, Assigned: protonk)
References
Details
Attachments
(1 file)
Display list of drafts within a thread (in a Drafts tab)
Long press of a draft gives option to edit or delete
Reporter | ||
Updated•11 years ago
|
Assignee: nobody → achyland
Blocks: messaging-drafts, 931095
Updated•11 years ago
|
Target Milestone: --- → 1.3 Sprint 5 - 11/22
Updated•11 years ago
|
Component: Gaia → Gaia::SMS
Assignee | ||
Comment 1•11 years ago
|
||
Cherry-picks rick's patch from Bug 935177
Attachment #8338791 -
Flags: review?(waldron.rick)
Comment 2•11 years ago
|
||
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
Comment 3•11 years ago
|
||
Comment 4•11 years ago
|
||
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)
Comment 5•11 years ago
|
||
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.
Description
•