926341 - [UX]To support WPA-EAP and EAP SIM options in WLAN setting

Status: RESOLVED FIXED

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

Description

[Ken Chang[:kenkai]]

We need to support other WPA-Enterprise methods. Here are the items.
1. To support PEAP, TLS, TTLS, and SIM for EAP.
2. To support PAP, MSCHAP, MSCHAPV2 for phase 2.

Comment 1

[Neo Hsieh [:neo]]

Attachment: Settings_WPA_EAP_SIM_131025.pdf

Hi All, I uploaded the UX specification document. Please check.

Comment 2

[Neo Hsieh [:neo]]

Attachment: Settings_WPA_EAP_SIM_131029.pdf

Please use new UX specification document. Thanks

Comment 3

[Neo Hsieh [:neo]]

Attachment: Settings_WPA_EAP_SIM_131029_fixed.pdf

Comment 4

[Francis Lee [:frlee]]

hi Ian,

since Neo has provided all UX spec, could you please take this bug?

Comment 5

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

Modified title to change this task as UX work. Gaia work is 926334

Comment 6

[[e-mail], [e-mail]]

Hi Neo,

As discussion offline, I think there is a little bit wired in spec. page 4, flow 1. The networks security type is incorrect for Mozilla Mobile. It should be belong to WPA-PSK type. So, I would like to suggest for making the spec. more clear for WPA-Enterprise flow(Using other network name in this case.). Thanks.

Comment 7

[Neo Hsieh [:neo]]

Hi Ian,  That is just a sample. I change wording for example.

Comment 8

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

Per discussion on Nov8, there will be UI change for combining WPA-Enterprise and EAP-SIM as WPA-EAP category. Target UX document release date is set to Nov12.

Comment 9

[Neo Hsieh [:neo]]

Attachment: Settings_WPA_EAP_SIM_131113.pdf

Comment 10

[Hubert Lu[:hlu] <[e-mail]>]

Add moztrap link:
[EAP-SIM]
https://moztrap.mozilla.org/manage/cases/?pagenumber=1&pagesize=20&sortfield=created_on&sortdirection=desc&filter-tag=2295

[WPA-EAP]
https://moztrap.mozilla.org/manage/cases/?pagenumber=1&pagesize=20&sortfield=created_on&sortdirection=desc&filter-tag=2304

Comment 11

[Neo Hsieh [:neo]]

Attachment: Settings_WPA_EAP_SIM_131121.pdf

Update the UX document.

Comment 12

[Neo Hsieh [:neo]]

Hi Matej Novak, Could you please help to check page 5's error message string ? Thanks :)

Comment 13

[Matej Novak [:matej]]

(In reply to Neo Hsieh from comment #12)
> Hi Matej Novak, Could you please help to check page 5's error message string
> ? Thanks :)

Happy to help. Could you put the string you'd like me to review into a bug comment and provide a little background to help me understand? Thanks.

Comment 14

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

Hi Matej,

Please see the two error messages in page5 of attachment 8335804 [details].
Those are for failure cases when end users manage the certificates used to certify WPA-Enterprise WiFi hotspots.

#1 
Title: Import certificate failed
Content: This situation may be caused by damaged certificate file or other unknown reasons.
#2
Title: Delete certificate failed
Content: This situation may be caused by storage issue or other unknown reasons.

Comment 15

[Matej Novak [:matej]]

Unfortunately I don't understand this use case well enough to be able to comment on it thoroughly, but it seems like these are screens that only an advanced user would see. As such, I think the language can remain more technical and less friendly than other messages, but I worry about calling out "unknown reasons."

Here's what I would recommend:

#1 
Title: Certificate import failed
Content: This may be due to a damaged certificate file, for example.

#2
Title: Certificate deletion failed
Content: This may be due to a storage issue, for example.

Comment 16

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

AKA/AKA' will not be enabled in v1.3 (disabled from GAIA), because of following reasons: 
1. we don't have the approach/device to test it
2. no clear requirements at this moment
Bug 944226 created for tracking.

Comment 17

[Neo Hsieh [:neo]]

Got it.

Comment 18

[Neo Hsieh [:neo]]

Attachment: Settings_WPA_EAP_SIM_131128.pdf

Thanks for your help! Matej Novak  :)    
Sorry for late reply. I updated the document. Please check.

Comment 19

[Matej Novak [:matej]]

Looks OK to me. Thanks.

Comment 20

[Jason Smith [:jsmith]]

The user story bug this blocks (bug 922930) has a target milestone of future, so this is not a committed feature for 1.3. Non-committed features for 1.3 should not block the release.

Comment 21

[[e-mail], [e-mail]]

Hi Neo,

Since the gaia patch is in reviewing process, we still have some question about the string.

1. P4, flow 2, the header "Manage Cert." , shall we use the Cert. to avoid string truncated issue? Or we have another string to description simply.

2. p4, flow 4, the description "Please enter the certificate name", could we change description via "Enter nick name"? (via reviewer concern)

Thanks.

Comment 22

[[e-mail], [e-mail]]

Also needInfo Matej for the string suggestion here. Thanks.

Comment 23

[Neo Hsieh [:neo]]

Hi Ian, 
1. Please use "Manage Cert." for consustency the button wording.
2. Sorry, I forgot to change it last time. I will prefer to "You can change the nick name or keep the default name."
I will update the document later.

Comment 24

[[e-mail], [e-mail]]

(In reply to Neo Hsieh from comment #23)
> Hi Ian, 
> 1. Please use "Manage Cert." for consustency the button wording.
> 2. Sorry, I forgot to change it last time. I will prefer to "You can change
> the nick name or keep the default name."
> I will update the document later.
Thanks for your input and check again. Will revise the string via your comment.

Comment 25

[Tim Guan-tin Chien [:timdream] (please needinfo)]

(In reply to Jason Smith [:jsmith] from comment #20)
> The user story bug this blocks (bug 922930) has a target milestone of
> future, so this is not a committed feature for 1.3. Non-committed features
> for 1.3 should not block the release.

Triage: AFAIK this is a commit feature, needinfo? Wesley to confirm.

Wesley, please either 1.3+ or --- this bug.

Comment 26

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

This bug is for UX document, which was provided already.