Skip to content
This repository has been archived by the owner on Mar 22, 2022. It is now read-only.

Fix toLowerCase hook #74

Closed
enten opened this issue Feb 23, 2016 · 1 comment · Fixed by #75
Closed

Fix toLowerCase hook #74

enten opened this issue Feb 23, 2016 · 1 comment · Fixed by #75

Comments

@enten
Copy link
Contributor

enten commented Feb 23, 2016

There is a camelcased mistake in src/hooks/to-lower-case.js (line 15):

// src/hooks/to-lower-case.js (line 15)
 if (obj[fieldName] && obj[fieldName].toLowercase) {

toLowercase isn't a method of String prototype: but toLowerCase is (thanks captain obvious!)

Do you want me to do a PR?
or can you fix and release that mistake quickly?

Thanks a lot for your great work!

@enten
Copy link
Contributor Author

enten commented Feb 23, 2016

I made a PR to save your time.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant