Make a Nightly build for user testing of Milestone 3 (M3)

RESOLVED FIXED

Status

()

P1
normal
RESOLVED FIXED
6 months ago
19 days ago

People

(Reporter: MattN, Assigned: MattN)

Tracking

(Depends on: 2 bugs)

unspecified
All
Windows
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox63 disabled, firefox64 disabled)

Details

(Whiteboard: [webpayments])

User Story

* Easy-to-use installer (self-extracting zip)
* Easy to uninstall
** Just delete the .exe, it gets extracted to a temporary folder so will get cleaned up automatically.
* Release branding
* Use a clean temp. profile with prefs:
** dom.payments.request.enabled=true (default now)
** dom.payments.request.user_interaction_required=false
** browser.search.region=US
** Disable some FTU distractions
*** browser.shell.checkDefaultBrowser=false
*** browser.contentblocking.enabled=false
* Windows-only
* Force official branding for card logos
* Fix bug 1501170
* Fix bug 1501162
* Preset jcrew.com bookmark
** browser.showPersonalToolbar=true

Nice to have:
* Signed binary to avoid Windows 8+ modal security warning?
** Otherwise "More Info" => "Run Anyway" need
* Use build with OS secure key storage

Attachments

(1 attachment)

We need a self-extracting build of Firefox Nightly for users to use for user testing of Milestone 3.

See https://github.com/bgrins/firefox-user-testing-helpers for some helpers, instructions and ideas.
Flags: qe-verify+
Whiteboard: [webpayments-reserve] → [webpayments]
Whiteboard: [webpayments] → [webpayments-reserve]
Whiteboard: [webpayments-reserve] → [webpayments]
Assignee: nobody → MattN+bmo
Status: NEW → ASSIGNED
User Story: (updated)
Priority: P2 → P1
User Story: (updated)
User Story: (updated)
Here is a version 1 build (for Windows) ready for review:
https://drive.google.com/file/d/1gHNjg5V-lwo8JbHgFHWkXx3xXlNgaVPX/view
User Story: (updated)
Attachment #9019252 - Flags: review?(sbautista)
status-firefox63: --- → disabled
status-firefox64: --- → disabled
I clicked through the build according to the research plan, and I noticed the following things. Can you please let me know if this is as intended?

When entering my billing address on the first screen for FTU, as I started typing, I saw a red error flash at the top of the modal. It might have been the "can't ship" error. I continued typing my address and the error did not re-appear. If this is a bug, I'm worried it might be off-putting to participants at the start of the interview.

Later in the flow, I also saw the invalid email error as I was typing my email address even though I wasn't done typing and had not moved on to another field.

I may have missed this in the design process, but is there a reason we're calling Preferences Firefox Options? None of the participants may notice, but once they're in Preferences, the address bar says Preferences, not Options.
Flags: needinfo?(jsavory)
Flags: needinfo?(epang)
Flags: needinfo?(brjones)
(In reply to sbautista from comment #3)
> I clicked through the build according to the research plan, and I noticed
> the following things. Can you please let me know if this is as intended?

Thanks for testing it out!

> When entering my billing address on the first screen for FTU, as I started
> typing, I saw a red error flash at the top of the modal. It might have been
> the "can't ship" error. I continued typing my address and the error did not
> re-appear. If this is a bug, I'm worried it might be off-putting to
> participants at the start of the interview.

If you mean shipping address, not billing address, then this is something I was also concerned about and was talking with UX about it last week but we thought it would be only visible if the user had an error so decided it was fine. Now I remember that it shows between the time the user clicks Next (which sets the selectedShippingAddress) and the time that the merchant responds with shipping options so it will show for everyone for a short amount of time. This is bug 1481243 but it depends on a DOM fix for the proper solution. I spent a while trying to figure out temporary solutions for the user test but didn't have much luck other than just hiding all generic errors on the credit card page.

> Later in the flow, I also saw the invalid email error as I was typing my
> email address even though I wasn't done typing and had not moved on to
> another field.

Hmm… that should only be the case if the field was already in the red invalid state before you focus the field. If not, can you file a bug blocking this one: https://bugzilla.mozilla.org/enter_bug.cgi?product=Firefox&component=WebPayments%20UI&blocked=1490809

> I may have missed this in the design process, but is there a reason we're
> calling Preferences Firefox Options? None of the participants may notice,
> but once they're in Preferences, the address bar says Preferences, not
> Options.

That is the name that we always use on Windows/Linux. If you look in the main menu it should match there. Yes, the URL always says about:preferences but that's not specific to Web Payments so it's probably fine for the test.
Depends on: 1481243
1. Sharon we talked about the error in our sync today, and will hide the error for the user study so people shouldn't see this. Unless they actually do have a shipping address that is not accepted and the merchant sends an error. 

2. Matt's understanding is correct, it should only appear if you task away from the field and then return to it. It shouldn't be appearing when you are just typing. 

3. I never knew this! I agree that we want what is in the UI, so Options makes sense for Windows and Preferences makes sense for Mac.
Flags: needinfo?(jsavory)
Flags: needinfo?(epang)
(In reply to Jacqueline Savory [:jsavory] UX from comment #5)
> 1. Sharon we talked about the error in our sync today, and will hide the
> error for the user study so people shouldn't see this. Unless they actually
> do have a shipping address that is not accepted and the merchant sends an
> error. 

I'm making a new build with those changes and I will provide the new link when it's ready.

Try push: https://treeherder.mozilla.org/#/jobs?repo=try&group_state=expanded&revision=879ed6451e38e25b69638a9a5c2a14db6ff7fc2d


> ** Otherwise "More Info" => "Run Anyway" need

Sharon, was the Windows 10 security dialog ok to work with?
Matt and Jacqueline: Thanks for the input. I will double check on the email error. And I learned something new about Preferences being called Options on Windows! 

The Windows 10 security dialog was fine. I noted the warning language for myself and will add a reminder in my discussion guide to reassure participants before that dialog comes up.
Flags: needinfo?(brjones)
Attachment #9019252 - Attachment description: Bug 1490809 - Make a build for user testing of Payment Request → Bug 1490809 - Make a build for user testing of Payment Request. r?jaws,sfoster
Here is a version 2 build with the flash of the shipping error addressed: https://drive.google.com/file/d/1_iOcNpxPJ97NeXAyidyQSX0v5IbI4vVC/view
Flags: needinfo?(sbautista)
Sharon noticed the lack of bug 1483412 in her test this time so I did a new try push with that cherry-picked on top of the same revision as version 2 so that I don't include bug 1429265 which needs more work.

https://treeherder.mozilla.org/#/jobs?repo=try&group_state=expanded&revision=91f6cdbbcdf12f3e2ad6d9896938de711557f061
Depends on: 1483412
Here is the version 3 build: https://drive.google.com/file/d/1fBXMcIA0XWrufvKTGnQLv1VrP0JszCrL/view

I'm going to mark this as resolved now but if you find issues feel free to comment.
Status: ASSIGNED → RESOLVED
Last Resolved: 5 months ago
Resolution: --- → FIXED
Version 4 build: https://drive.google.com/file/d/1oRaOs_O1ACuq1TB87Ln7vOXKj5gQEQ69/view

The only changes are in the temp. profile:

+user_pref("dom.payments.response.timeout", 20000);
+user_pref("browser.crashReports.unsubmittedCheck.enabled", false);

These are based upon things noticed in the first two tests.
Flags: needinfo?(sbautista)
Comment on attachment 9019252 [details]
Bug 1490809 - Make a build for user testing of Payment Request. r?jaws,sfoster

We have been using this fairly successfully for 3 participants already.
Attachment #9019252 - Flags: review?(sbautista) → review+
Here are the revisions used for version 5:

o   443745:c45e8f2204c9 mozilla
|  Bug 1490809 - Make a build for user testing of Payment Request. r?jaws,sfoster
o   443744:75e9f4d8f4fd jwein
|   Bug 1483412 - Make the telephone number field required on the payer form if requestPayerPhone is true. r=MattN
o     443743:d71404ed02ef aiakab
|\   Merge inbound to mozilla-central a=merge
Whiteboard: [webpayments] → [webpayments] [qa-triaged]
QA Whiteboard: [qa-triaged]
Whiteboard: [webpayments] [qa-triaged] → [webpayments]

Comment 15

19 days ago

Removing the qe-verify+ flag since the feature is disabled.

Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.