3075 add historical context #3344
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Deploy preview for redux-docs ready! Built with commit 0de4915 |
|
Great, thanks! Busy with some other things atm, but I'll make sure to come back and look at this shortly. |
|
Thanks Mark!
…On Mon, Jan 28, 2019, 1:39 PM Mark Erikson ***@***.***> wrote:
Great, thanks! Busy with some other things atm, but I'll make sure to come
back and look at this shortly.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#3344 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAPechok5zYY00oIHjW3GlpRu1nr2K6Nks5vH24NgaJpZM4aW0vM>
.
|
|
Hi, just wanted to ping.
…On Mon, Jan 28, 2019 at 2:04 PM Monica Tie ***@***.***> wrote:
Thanks Mark!
On Mon, Jan 28, 2019, 1:39 PM Mark Erikson ***@***.***>
wrote:
> Great, thanks! Busy with some other things atm, but I'll make sure to
> come back and look at this shortly.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <#3344 (comment)>, or mute
> the thread
> <https://github.com/notifications/unsubscribe-auth/AAPechok5zYY00oIHjW3GlpRu1nr2K6Nks5vH24NgaJpZM4aW0vM>
> .
>
|
|
Hah, sorry, looks like I forgot this PR was here :) Ah... glancing at it, the immediate issue is that there's no pointer to this in the docs sidebar, so it's not showing up anywhere. Can you add an entry to put it under the "Other" category for now? |
|
Oh! Sure thing, I'll submit an update
…On Wed, Oct 16, 2019, 2:44 PM Mark Erikson ***@***.***> wrote:
Hah, sorry, looks like I forgot this PR was here :)
Ah... glancing at it, the immediate issue is that there's no pointer to
this in the docs sidebar, so it's not showing up anywhere. Can you add an
entry to put it under the "Other" category for now?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#3344?email_source=notifications&email_token=AAB544V77SFIRK6SLUTURL3QO6DM3A5CNFSM4GS3JPGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBOA64Q#issuecomment-542904178>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAB544UEMKM4A7IUMMH4DH3QO6DM3ANCNFSM4GS3JPGA>
.
|
|
Ah I vaguely remembered this context coming up in relation to the Motivation and Prior Art intro chapters, so I grouped it right before Prior Art to make it chronological and transition into Prior Art. |
|
Tell you what. We're about to start a major revamp and reorganization of the Redux core docs (see #3313 ). I'm going to hold off on merging this for the moment. Once we've nailed down what the revised docs structure should be, we can slot it in with the rest of the changes. |
|
Sounds good!
…On Thu, Oct 17, 2019 at 10:06 AM Mark Erikson ***@***.***> wrote:
Tell you what. We're about to start a major revamp and reorganization of
the Redux core docs (see #3313
<#3313> ). I'm going to hold off
on merging this for the moment. Once we've nailed down what the revised
docs structure should be, we can slot it in with the rest of the changes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#3344?email_source=notifications&email_token=AAB544TTBLY572NG2Z7AU2TQPCLSHA5CNFSM4GS3JPGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBQ2OXY#issuecomment-543270751>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAB544U2USJA2UX4XAQWNBDQPCLSHANCNFSM4GS3JPGA>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi! I took a stab at issue 3075 (#3075)
I read over Mark's links and docs and tried to condense as much of the information into a factual, easily digestible format and limited it to core context pieces. I definitely cut a lot of content in doing so, and can imagine adding some back for more info.
I decided to limit the historical context to three things: Flux, influence of functional paradigms and a philosophy of keeping things simple. These were the main influences I found to have the most historical impact on the React/JS community over the past few years and in my own year long investigation into comparing/contrasting state management libraries in my professional work.
I hope this at least gives a starting point to adding Historical Context for those looking to understand the background. Looking forward to discussing it with you all and making updates.