Closed Bug 1105319 Opened 11 years ago Closed 11 years ago

We don't allow other donation values with a decimal in it, example 2.00

Categories

(Mozilla Foundation Communications :: Website, task)

x86
macOS
task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: becky, Assigned: mavis)

References

Details

(Whiteboard: [EOYFR2014][donation flow][nov28] [p2])

Attachments

(2 files)

+++ This bug was initially created as a clone of Bug #1089403 +++ Build sequential donation form for EOY2014 Scott ---- I entered "2.00" as my donation amount, and I got an error. When I enter "2", it works. I've attached a shot of the error. Hope it helps.
Summary: Build Sequential Donation form for EOY2014 → We don't allow other donation values with a decimal in it, example 2.00
Okay. I feel like a lot of donors might try to enter a number with a decimal. Hopefully not.
I thought this would be easy, but looks like there is a bug in parsley and confusion between html5 input types and parsley. https://github.com/guillaumepotier/Parsley.js/issues/508 Maybe an update would fix it? Otherwise, more debugging is needed.
Depends on: 1105474
No longer depends on: 1095842
No longer depends on: 1095868
No longer depends on: 1095835
No longer depends on: 1096838
Assignee: scott → mavis
Incorrect dependency here. :)
Whiteboard: [EOYFR2014][donation flow][nov14] [p2] → [EOYFR2014][donation flow][nov28] [p2]
Parsley auto-binds integer validator to HTML type="number" fields (see http://parsleyjs.org/doc/#psly-validators-list) and we can't override that. There are 2 options we can go with, each with some pros and cons 1) do NOT use HTML type="number" pros: we can accept non-integer number cons: on mobile, the number pad won't pop up automatically when the "Other" field is in focus. Users have to switch to the number pad manually 2) continue using HTML type="number" pros: on mobile, number pad pops up automatically. This saves users a step from entering the number. cons: we can only accept integer in the "Other" field thoughts?
Flags: needinfo?(scott)
Flags: needinfo?(becky)
Flags: needinfo?(andrea)
#2 is probably better, if most people are using it on mobile.
Flags: needinfo?(becky)
There's no way to decide this accurately without data. More than 95% of our donor traffic is desktop, so because of that I'm inclined to say #1, but I'm adding Adam to see if there's an easy way to test this.
Flags: needinfo?(andrea) → needinfo?(adam)
Yeah, desktop should be the focus. Donations of 3.50 that get rejected are more likely to either bail, or become 3.00, costing us money. Where a mobile donation key pad can be toggled between text and numbers. Let's go with 1.
Flags: needinfo?(scott)
Decision: let's go for option #1
Flags: needinfo?(adam)
See Also: → 1105882
Attached file GitHub PR
Attachment #8532168 - Flags: review?(scott)
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Attachment #8532168 - Flags: review?(scott) → review+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: