-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Missing credentials #26
Comments
Try to execute something like before submitting the form (for testing purpose): |
I resolved with |
It looks like autofill should trigger a change event after filling the fields (its FF issue, Chrome should works). Since it does seem like derbyjs is watching for the change event. |
ok. at this point my solution is only a quick work around Btw, I saw a similar problem with another form (not in derby-login), where I have a |
Solved by #27 |
Hi,
I found a strange behaviour when my login is autofill by the browser.
Basically when the page is loaded and the browser (firefox 35.0.1) fills the user and password for me.
If I click the
Login
button right away the "missing credentials" error is given to me.On the server side I receive
email
andpassword
asundefined
.I believe this is because the model didn't received the input by the user, so
data[field] = this.model.get(field)
in thebase
class returnsundefined
.The strange thing is that if I than remove both fields, start typing the user and select the one I want from the browser saved credentials dropdown, and the browser fills up the fields for me, then I can succesfully login - even though I didn't filled up the password at all, and I partially filled the email field.
To solve this behaviour I tried to get the field data directly from the dom, with somehting like
data[field] = this.model.get(field) || this.dom.getElementById(field).value
, but there's no such function, as I didn't find other functions which allows me to get a dom element refence from thedom
object inside a component.Any Idea? Is it happening only to me?
Thanks a lot!
Cheers
The text was updated successfully, but these errors were encountered: