newslynx / opportunities

A garden of NewsLynx futures
1 stars 0 forks source link

app should strip input in login form #164

Open abelsonlive opened 8 years ago

abelsonlive commented 8 years ago

this was a confusing error:

screenshot 2015-10-06 13 22 07

i guess you posted a form with json like:

{
   "email": " changeme@example.com",
   ...
}
mhkeller commented 8 years ago

when did this happen? i could modify the ui to automatically trim as you type but modifying the data without giving the user feedback could be problematic.

On Tue, Oct 6, 2015 at 1:24 PM, Brian Abelson notifications@github.com wrote:

Assigned #164 https://github.com/newslynx/opportunities/issues/164 to @mhkeller https://github.com/mhkeller.

— Reply to this email directly or view it on GitHub https://github.com/newslynx/opportunities/issues/164#event-428225036.

@mhkeller mhkeller.com

abelsonlive commented 8 years ago

when i was testing.