Closed Bug 842251 Opened 11 years ago Closed 11 years ago

[Settings][Message] Configurable requisition for delivery report at sending messages

Categories

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

x86
macOS
defect
Not set
normal

Tracking

(blocking-b2g:leo+, b2g18 fixed, b2g18-v1.0.0 wontfix, b2g18-v1.0.1 wontfix)

RESOLVED FIXED
blocking-b2g leo+
Tracking Status
b2g18 --- fixed
b2g18-v1.0.0 --- wontfix
b2g18-v1.0.1 --- wontfix

People

(Reporter: khu, Assigned: steveck)

References

Details

Attachments

(1 file)

Gecko part should be ready, but need to confirm if Gaia integrated this or not. Yuren, can you help to confirm? Thanks!
Since the requirement is to have it OFF by default which meets current status, I will close this case for now. Thanks.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
We should have configurable send and requisition for delivery report now, but 849112 is still ongoing. Reopen this bug as gaia settings feature and for sms delivery report only. Hi Vicamo, is there any related gecko bug for sms delivery report ?
Assignee: yurenju.mozilla → schung
Status: RESOLVED → REOPENED
Flags: needinfo?(vyang)
Resolution: FIXED → ---
blocking-b2g: --- → leo+
Blocks Gecko part bug 863130.
Blocks: 863130
Flags: needinfo?(vyang)
Summary: SMS delivery report → [SMS] Configurable requisition for delivery report at sending messages
s/Blocks/Blocked by/. Sorry for the noise.
Component: Gaia::SMS → Gaia::Settings
Summary: [SMS] Configurable requisition for delivery report at sending messages → [Settings][Message] Configurable requisition for delivery report at sending messages
Attachment #747233 - Flags: review?(arthur.chen)
Hi Arthur,
If you need the wireframe, please ref:
https://www.dropbox.com/s/mrtjwvzk6xgoc02/HTML5_SMS-MMSUserStorySpecifications_20130503_V8.0.pdf
Thanks.
Comment on attachment 747233 [details]
Add delivery report option in settings app

Thank you for the work, Steve. I have a concern on the newly added style. There is already existing style doing the same thing. Also, the implementation is slightly different than the spec. Details please refer to github, thanks!
Attachment #747233 - Flags: review?(arthur.chen)
Comment on attachment 747233 [details]
Add delivery report option in settings app

The updated patch looks good. r=me. Thanks!
Attachment #747233 - Flags: review+
(In reply to Arthur Chen [:arthurcc] from comment #7)
> Comment on attachment 747233 [details]
> Add delivery report option in settings app
> 
> Thank you for the work, Steve. I have a concern on the newly added style.
> There is already existing style doing the same thing. Also, the
> implementation is slightly different than the spec. Details please refer to
> github, thanks!

I replied your concern in github :https://github.com/mozilla-b2g/gaia/pull/9633#discussion_r4149106
We can discuss about this next week in Taipei, thanks.
Merged in master: 65a0f833064dbff4efc8210c86291897e9a9724e
Status: REOPENED → RESOLVED
Closed: 11 years ago11 years ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Status: REOPENED → RESOLVED
Closed: 11 years ago11 years ago
Resolution: --- → FIXED
Uplifted 65a0f833064dbff4efc8210c86291897e9a9724e to:
v1-train: eb780695a5c6771b0a16707d4aed0900070cbc80
Blocks: 877148
Flags: in-moztrap-
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: