Skip to content
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

Add derivation of thunk #150

Merged
merged 1 commit into from
May 25, 2018
Merged

Add derivation of thunk #150

merged 1 commit into from
May 25, 2018

Conversation

dthree
Copy link
Contributor

@dthree dthree commented Aug 18, 2017

Thunks have always confused me, and this is solely due to the fact that I never fully understood the word. Just knowing the specialized sense of a word does not clear up confusion when you have no idea where the word came from.

I have always been asking "but why is it called thunk?" This has honestly blown me off of Redux entirely for two years. I finally took the effort to find out.

I'm proposing we add the derivation of the term and its source, which will help greatly in clearing up understanding for people.

Thunks have always confused me, and this is solely due to the fact that I never fully understood the word. Just knowing the specialized sense of a word does not clear up confusion when you have no idea where the word came from.

I have always been asking "but *why* is it called thunk?" This has honestly blown me off of Redux entirely for two years. I finally took the effort to find out.

I'm proposing we add the derivation of the term and its source, which will help greatly in clearing up understanding for people.
@edwardkcyu
Copy link

edwardkcyu commented Feb 20, 2018

I also had the same question for a long time, finally got the answer in this PR. I think other people may have the same question too.

@timdorr
Copy link
Member

timdorr commented May 25, 2018

Thanks!

@timdorr timdorr merged commit 8bcd7c7 into reduxjs:master May 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants