search

rich-at-thinkful / sample-quiz-app-oo

0 stars 1 forks source link

Request: Rename fetchToken to getToken #2

Open cklanac opened 6 years ago

cklanac commented 6 years ago

Could you rename fetchToken to getToken or something similar which doesn't use "fetch" just so it doesn't get confused with functions which use the Fetch API?

https://github.com/rich-at-thinkful/sample-quiz-app-oo/blob/master/app.js#L25

rich-at-thinkful commented 6 years ago

At the time I think I liked the nomenclature of fetch for remote retrieval and get for local getters, but not married to it. This may be moot anyway because this is likely being sunset if current week 2 overhaul remains (Quiz App is out of EI curric...).