osuosl / formsender

Simple script to email form submissions.
Apache License 2.0
3 stars 1 forks source link

Changed instances of 'hidden' to 'last_name' #24

Closed matthewrsj closed 9 years ago

matthewrsj commented 9 years ago

Updated docs, tests, and app

Kennric commented 9 years ago

It looks like you changed tests and code all at once. That's fine, but normally we would change the tests to reflect the new field name, review, and then implement to pass the tests. This looks fine to me, though, I don't see any issues and all names appear to have been changed correctly. +1