search

ga-wdi-boston / jquery-ajax-get-element

Other
0 stars 82 forks source link

require getFormFields #2

Closed raq929 closed 7 years ago

raq929 commented 8 years ago

This will get rid of a linter error and prepare us for: ga-wdi-boston/browser-template#49

jrhorn424 commented 7 years ago

1f3410042e2dd3530a2a9a2a109e8c48648c3c57

danman01 commented 7 years ago

This is present in the solution branch as @jrhorn424 mentioned in that commit link: https://github.com/ga-wdi-boston/jquery-ajax-get-item/blob/solution/assets/scripts/books/events.js#L5

Can we close this now or is there more needed?

gaand commented 7 years ago

@raq929 Can you comment? What was your intent? Did @jrhorn424 's commit fulfill it? Does the talk need scaffolding so whomever delivers it knows what to do?

raq929 commented 7 years ago

Yes. We used to be including it in index.html or something strange. I don't remember what. Jeff's commit handles it.