[Bluetooth] Refine the description of header/body while the storage is not available



Firefox OS
4 years ago
4 years ago


(Reporter: iliu@mozilla.com, ianliu.moz@gmail.com, Assigned: iliu@mozilla.com, ianliu.moz@gmail.com)


Firefox Tracking Flags

(Not tracked)



(1 attachment)

The issue comes following Bug 915584, section description, error message. Error message improving issue.

* Use the device without a SD Card, from another device send a file via bluetooth to it.
* There is a prompt for guiding user how to solve the storage problem.

header: 'no memory card'
body: 'Insert a memory card to transfer files using BT'

header: 'Can not receive file'
body: 'No memory card found (Please insert a memory card)'
According to Bug 915584, comment 9(https://bugzilla.mozilla.org/show_bug.cgi?id=915584#c9), storage status are as following: 
=== Storage Status ===
Can not get device storage state
No memory card found (Please insert a memory card) #here we're modifying.. 
Bluetooth can not be used while phone is plugged in. Please unplug the phone to receive file(s).
Can not complete transfer: memory card is full.
Unknown error

If we want to revise the general header 'Can not receive file', we have to considerate each storage status. Could UX give some input here? Thanks.
Flags: needinfo?(cawang)
Flags: needinfo?(aymanmaat)

Comment 2

4 years ago
i will let carrie follow up on this. ni? me if you require any input.
Flags: needinfo?(aymanmaat)

Its been a while since the bug was last commented. Not sure if Carrie is traveling or out, please reassign appropriately.
Flags: needinfo?(mtsai)

Comment 4

4 years ago
Carrie will give a solution on this before the end of the week. Sorry for the late reply.
Flags: needinfo?(mtsai)

Comment 5

4 years ago
Ian wasn't in the office until this week. Carrie has talked about this bug before. Ian will sync with Carrie this week: )
Should need some background knowledge from engineering side. I'll discuss with Ian this week.
Flags: needinfo?(cawang)
Hi guys,

I'd suggest that we apply the same header with different body for different contexts.
So that would be:

header: 'Can not receive file'
1.'Can not get device storage state.'
2.'No memory card found. Please insert a memory card.'
3.'Bluetooth can not be used while phone is plugged in. Please unplug the phone to receive file(s).'
4.'Can not complete transfer: memory card is full.'
5.'Unknown error'

moving to 1.3
blocking-b2g: koi? → 1.3?
Created attachment 827228 [details]
Pointer to Github pull request 13372.html

This is a very small patch for revising the description of body.

Hi Stas,

Could you please help to review the pr? Thank you.
Attachment #827228 - Flags: review?(stas)


4 years ago
Keywords: late-l10n

Comment 10

4 years ago
removing late-l10n, as this is not for koi.
Keywords: late-l10n

Comment 11

4 years ago
Comment on attachment 827228 [details]
Pointer to Github pull request 13372.html

Nit, if you just paste 'https://github.com/mozilla-b2g/gaia/pull/13372' into the attachment context, you get bugzilla to do the redirect magic for you.

giving this an r-, even though neither stas nor I are good reviewers for this.

Just adding the '.' to an existing string might pass as a typo fix, but the removal of the () sounds like a semantic change that should be pushed out to localizers by changing the ID.
Attachment #827228 - Flags: review?(stas) → review-
Hi Axel,

Thanks for your reminder with changing the string ID. I have revised the patch according to your guidance. Could you suggest a reviewer for it? Or the Gaia::system module owner is fit to review the locale issue in v1.3 implementing stage. Thank you so much.
Flags: needinfo?(l10n)

Comment 13

4 years ago
Your usual review suspects should review this code, and also find issues like this.

I'll also send out a reminder to .gaia.
Flags: needinfo?(l10n)
Comment on attachment 827228 [details]
Pointer to Github pull request 13372.html


Could you please help to review the string refinement patch? Thanks.
Attachment #827228 - Flags: review?(alive)
Attachment #827228 - Flags: review?(alive) → review+
Since the pr is merged, we can close the issue now. Thanks for Alive's reviewing effort.
Last Resolved: 4 years ago
Resolution: --- → FIXED
Already in 1.3, clearing nom.
blocking-b2g: 1.3? → ---
You need to log in before you can comment on or make changes to this bug.